-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[UI] Dropdown menus #13749
base: 5.x
Are you sure you want to change the base?
[UI] Dropdown menus #13749
Conversation
revert Revert "fixing broken darken/lighten" This reverts commit bcf4ac98e07c0c17162a74794a8161d5398c3d7c.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 5.x #13749 +/- ##
============================================
+ Coverage 61.62% 61.93% +0.30%
- Complexity 34145 34195 +50
============================================
Files 2245 2248 +3
Lines 102077 102265 +188
============================================
+ Hits 62909 63337 +428
+ Misses 39168 38928 -240 |
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.
Works and looks good on both desktop and mobile 👍
Isn't this PR based on #13735? |
@andersonjeccel as mentioned in other similar PR's, this is another one based on the Tokens branch. I think there's 5 or 6 now? Please put it in draft until we discussed what you're trying to achieve and set a better course of action. |
bdce257
to
2a12ac5
Compare
Description
This PR is based on another PR, where I introduce tokens to replace most of the UI variables, aiming to improve consistency in all components. We need it to be merged before this one becomes available for code review.
It changes dropdown styles and makes it compatible with dark theme.
Also fixes the too tiny size of clickable items on mobile.
Now when you hover over danger actions, have better visual feedback:
dropdown-danger.mp4
Or navigating with your keyboard arrows
dropdown-focus.mp4
📋 Steps to test this PR: