Skip to content

Improved reply indicators#642

Open
qasimstatic wants to merge 4 commits intoayn2op:mainfrom
qasimstatic:feature/theme-based-reply-indicators
Open

Improved reply indicators#642
qasimstatic wants to merge 4 commits intoayn2op:mainfrom
qasimstatic:feature/theme-based-reply-indicators

Conversation

@qasimstatic
Copy link
Copy Markdown

@qasimstatic qasimstatic commented Dec 1, 2025

Summary

  • Replaced '>' with '󱞩' reply indicator(nf-md-arrow_right_bottom)
  • Move indicator from referenced message to actual reply
  • Properly dim referenced messages for better visual hierarchy
  • Update forwarded indicator(nf-md-arrow_right_top) to matching '󱞫' icon

Changes

  • Modified message rendering logic in cmd/messages_list.go
  • Updated default indicators in internal/config/config.toml
  • Improved visual distinction between referenced messages and actual replies

Testing

  • Verified reply indicators appear before actual reply messages
  • Confirmed referenced messages are properly dimmed
  • Tested with various message types (replies, regular messages, forwarded)

Summary by CodeRabbit

  • New Features

    • Enhanced message reply display with improved formatting and positioning of reply indicators.
  • Style

    • Updated reply and forwarded message indicators to Unicode arrows (↳ and ↱) for clearer visual distinction.

✏️ Tip: You can customize this high-level summary in your review settings.

- Replaced '>' with '󱞩' reply indicator
- Move indicator from referenced message to actual reply
- Properly dim referenced messages for better visual hierarchy
- Update forwarded indicator to matching '󱞫' icon
@xqrs
Copy link
Copy Markdown
Collaborator

xqrs commented Dec 1, 2025

image The character does not render for me, and there's no search result for it.

@qasimstatic
Copy link
Copy Markdown
Author

would you mind checking
nf-md-arrow_right_bottom
nf-md-arrow_right_top
here

@xqrs
Copy link
Copy Markdown
Collaborator

xqrs commented Dec 1, 2025

It's a nerd-font specific glyph, huh. I don't think @ayn2op would like that.

@qasimstatic
Copy link
Copy Markdown
Author

@xqrs Yeah thank you for telling me that I also assumed that would be the case so we could use the usual unicode '↱' and '↳'

@ossenthusiast
Copy link
Copy Markdown
Contributor

image

This looks rather odd, imo. Contrary to the official client, the indicator is now after the timestamp, so it is not immediately obvious to the user that the message is a reply.

Screenshot 2025-12-01 at 2 17 06 PM

@qasimstatic
Copy link
Copy Markdown
Author

Oh yeah I am trying to get the hierarchy in which discord displays their messages.
If one person stacks 3 messages under his message his name doesn't repeat.
Mb I was testing with time off because of this.
I'll make some edits for the time and reply indicator to check if it looks good

Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 2, 2025
Repository owner deleted a comment from coderabbitai bot Dec 3, 2025
Copilot AI review requested due to automatic review settings December 8, 2025 23:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants