-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(slimbar): create unread user component #2475
Conversation
✅ Yeeeehaw, deploy preview is ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Tested: Should this be functional? I do get sound feedback but when I try to use the quick menu doesnt seem to be functional right now functional.mov |
I don't know what functional exactly should be in context menu. Maybe @molimauro knows? |
Can you replicate the same in the friends list in the sidebar? |
fixed |
add active state for selected unread and revert sound effect place |
Thanks @AlexZakablukov great job |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ohh lol I thought it wasn't the same 😂 thanks for checking @AlexZakablukov I thought the colored ones were fake hidden chats lol, I'm gonna add a ticket for those to be grey out then 🔨 |
@stavares843 I can disable hover effect on mobile and leave only active state. Would it be a good solution? Also I think I have to move tooltip with name to bottom or remove it on mobile |
sounds good @AlexZakablukov removing the hover tip on mobile seems a good idea |
fixed |
What this PR does 📖
create unread user component that display the new messages count, play sounds, show a custom context menu on mouse right click, show tooltip when hovered
Which issue(s) this PR fixes 🔨
AP-1141
Special notes for reviewers 🗒️
Additional comments 🎤