-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[joy-ui][docs] Add UI improvements to the side navigation demo #41461
Conversation
The right sidebar has been updated to the ‘solid’ variant for improved interaction with IconButtons. It is now darker than the left sidebar, creating better relative contrast and lifting the left sidebar higher on the screen. Additionally, the styles are consistent whether in light or dark mode, addressing previous issue where the right sidebar appeared higher/lower based on the color scheme.
1. **Avatar Visibility Improvement:** Previously, when the avatar contained a picture, the interaction was hidden. But now, it's fixed, You’ll see the hover, active, and all the props that we expect from a button. (tested on dark/light mode). 2. **Plus Icon Enhancement:** The plus icon shape used to look distorted. We’ve standardized the size for all icon buttons in the right sidebar. Additionally, we’ve ensured that they are neatly aligned using align-items.
If there are several users online, it’s a good idea to use the color ‘success’ for all badges with `color="success"`. ps: the badgecolorsuccess import was removed, it isn't needed.
Netlify deploy previewhttps://deploy-preview-41461--material-ui.netlify.app/ Bundle size report |
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.
Great PR — thanks for refining this further! My comments are mostly just fine-tuned tweaks. 😃
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
docs/data/joy/main-features/color-inversion/ColorInversionNavigation.js
Outdated
Show resolved
Hide resolved
Thank you, @danilo-leal I’ve applied all the modifications except for '--IconButton-size': '3rem'. When Don't you think that the if normal state:hover state:active state:Minor changesI made a small change to the Plus Icon and the palette icon. They now have the text.tertiary color to blend well with the background. I also removed the aria-label from the plus icon button and wrapped it with a Tooltip. This helps users understand the button’s purpose and automatically adds an aria-label(win-win). |
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.
Looking great! Agree with your point there about Joy's Icon Button; an issue would be appreciated, even though we might take a while to get to it :)
Several enhancements to the side navigation in this pull request.
https://deploy-preview-41461--material-ui.netlify.app/joy-ui/main-features/color-inversion/#side-navigation
Here’s a concise overview:
1. Fixing IconButton Interactions:
2. Avatar Interaction Enhancement:
3. Specific Fixes and Modifications:
Before
After
Before
After
Other changes are explained withing each commit message.