-
Notifications
You must be signed in to change notification settings - Fork 154
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
[POC] Change light theme #1048
base: main
Are you sure you want to change the base?
[POC] Change light theme #1048
Conversation
for more information, see https://pre-commit.ci
--right-side-bg: var(--surfaces-bg02); | ||
--left-side-bg: var(--surfaces-bg01); |
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.
@Joseph-Perkins - this would be the only change required in terms of CSS now to switch out the bg colors from the outer and inner containers.
I'll comment on the issues in the PR so we can come back to it if we want
View the example dashboards of the current commit live on PyCafe ☕ 🚀Updated on: 2025-03-04 22:14:13 UTC Compare the examples using the commit's wheel file vs the latest released version: vizro-core/examples/scratch_devView with commit's wheel vs View with latest release vizro-core/examples/dev/View with commit's wheel vs View with latest release vizro-core/examples/visual-vocabulary/View with commit's wheel vs View with latest release vizro-ai/examples/dashboard_ui/ |
Description
If we were to implement this change - this is the default:

Charts on a light grey background
Change needed - card colors need to change to white, otherwise they blend in with the new background

Screenshot
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":