The-Commit-Company / raven

Simple, open source team messaging platform
https://thecommit.company/products/raven
GNU Affero General Public License v3.0
291 stars 117 forks source link

feat: added reaction-analytics button #1011

Closed pranavmene2000 closed 1 month ago

pranavmene2000 commented 1 month ago

reaction![Capture]

In case of multiple users: (https://github.com/user-attachments/assets/7c4c0d6b-af89-4ac8-bad2-e67f0c7b7f3a)

janhvipatil commented 1 month ago

@pranavmene2000 thank you for your PR, just a few suggestions here -

The count over here should be 5 right? (https://github.com/user-attachments/assets/7c4c0d6b-af89-4ac8-bad2-e67f0c7b7f3a)

Also, rather than having count as a separate column, can we combine the emoji and count in one column like so -

Screenshot 2024-08-12 at 8 51 37 PM

Move the show all reactions button right next to the reactions rather than at the flex end and make it grey, as it is not a primary action.

pranavmene2000 commented 1 month ago

@janhvipatil Yes. This is just an example. I edited dom to add users, so that I can take a screenshot for an example. But anyway, it will give the correct count as per the number of users.

janhvipatil commented 1 month ago

Oh okay, also do we have a scroll inside if the users are over 1000?

pranavmene2000 commented 1 month ago

Oh okay, also do we have a scroll inside if the users are over 1000?

@janhvipatil Inner Scroller: reaction-scroller