-
Notifications
You must be signed in to change notification settings - Fork 7
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: new light theme / inset theme for light and black theme #1288
Conversation
|
✅ Deploy Preview for harness-design ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for harness-xd-review ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
9c8dac4
to
36fbacc
Compare
6abe923
to
6f17bbe
Compare
6f17bbe
to
ed8b1f7
Compare
2c0bfd0
to
1b9f4d3
Compare
1b9f4d3
to
9455319
Compare
Sidebar Components Styling and Theming Update
Key Changes
Sidebar and Navigation
Theming
useTheme
hook across sidebar components for consistent theme handlingshared-style-variables.css
Components
isLightTheme
variant for proper theme-based renderingOptimization
Technical Details
cn
utility for conditional class applicationisInset
andisLightTheme
Impact
These changes improve the overall UI consistency and provide better theme support across the application. The new styling system makes the interface more maintainable and visually cohesive, especially in light theme mode.
Testing Notes
Related Changes
These updates enhance the user experience while maintaining the application's visual hierarchy and improving code maintainability.