-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
fix: modified css module #7570
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: modified css module #7570
Conversation
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.
Copilot wasn't able to review any files in this pull request.
Files not reviewed (1)
- packages/ui-components/Containers/Sidebar/SidebarItem/index.module.css: Language not supported
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Does this also fix the bug in the logo? |
Lighthouse Results
|
Nop I didn't get how to fix logo issue. Because dark still Is correctly applied on dark theme and correctly remove on light theme. But it's didn't applied light style. |
I think we pass to the logo what theme we are at or something? @avivkeller if you can investigate as a follow-up 🙏 |
We define it by tailwind classes
|
I'm fast tracking because it's affect a lot user |
Yep! I'll be on that ASAP. |
Thanks 🫶🏻 |
Okay. I'm so sorry about the UI differences, I'm currently looking into it and will open a PR as soon I as complete my investigation. |
Description
The css-module was modified I just copy the previous one. Also remove local in pathname to fix active link detection.
Validation
Related Issues
Close #7569
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.