Skip to content
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

fix(core): change design for clickable present users in dropdown #4154

Merged

Conversation

ninaandal
Copy link
Contributor

@ninaandal ninaandal commented Feb 3, 2023

Description

The LinkIcon for clickable users in the presence menu has been replaced with a new design. The users that are not clickable are now greyed out and has text under. Users that can be clicked on, are not.

Screenshot 2023-02-06 at 11 24 05

What to review

Click on the presence menu and make sure users that have a presence can be clicked on.

Notes for release

Design change of the presence menu where non-clickable users are greyed out

@vercel
Copy link

vercel bot commented Feb 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 7, 2023 at 10:17AM (UTC)
2 Ignored Deployments
Name Status Preview Comments Updated
performance-studio ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:17AM (UTC)
studio-workshop ⬜️ Ignored (Inspect) Feb 7, 2023 at 10:17AM (UTC)

@ninaandal ninaandal force-pushed the bug/sc-29587/edit-avatar-pull-down-to-show-clickable-avatars branch from 0244b03 to 701bd7f Compare February 6, 2023 11:01
Copy link
Contributor

@mikolajdobrucki mikolajdobrucki left a comment

Choose a reason for hiding this comment

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

lgtm! 🚀

@ninaandal ninaandal merged commit 81c9d4c into next Feb 7, 2023
@ninaandal ninaandal deleted the bug/sc-29587/edit-avatar-pull-down-to-show-clickable-avatars branch February 7, 2023 11:40
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.

3 participants