Skip to content

Show reactions - #181

Open
ln-001 wants to merge 4 commits into
subpop:mainfrom
ln-001:feature/show-reactions
Open

Show reactions#181
ln-001 wants to merge 4 commits into
subpop:mainfrom
ln-001:feature/show-reactions

Conversation

@ln-001

@ln-001 ln-001 commented Aug 30, 2026

Copy link
Copy Markdown

Instead of generic gray logo, show up to 3 emojis if reactions are added to a message.
Above 3 reactions you can click to expand and you can see how many people issued each reach reaction.
By hovering over the number you can also see who.

ln-001 added 2 commits August 30, 2026 14:41
…sage. If more than 3 are present, show a buton with the number of hidden emojis. Clicking on that will reveal them.
… than maxShown then the popover will show a "and X more" message at the end.
@ln-001
ln-001 force-pushed the feature/show-reactions branch from 6219f46 to 939d625 Compare August 30, 2026 12:43

@subpop subpop left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for your contribution! This is just a quick review with some thoughts I had on the code. I'll pull the branch and run it to see how it feels to me. This looks great from the screenshots!

Comment thread Relay/Views/Message/MessageReactionBadges.swift Outdated
Comment thread Relay/Views/Message/MessageReactionBadges.swift Outdated
Comment thread Relay/Views/Message/MessageReactionBadges.swift Outdated
@ln-001
ln-001 requested a review from subpop August 31, 2026 08:26
Comment thread Relay/Views/Message/MessageReactionBadges.swift Outdated
Comment thread Relay/Views/Message/MessageReactionBadges.swift Outdated
}
.padding(40)
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This line too. Should this be deleted?

Comment thread Relay/Views/Message/MessageView.swift Outdated
@ln-001
ln-001 requested a review from subpop September 2, 2026 20:06
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.

2 participants