-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 Dev UI Theme switch #42310
Fix Dev UI Theme switch #42310
Conversation
This comment has been minimized.
This comment has been minimized.
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.
LGTM
Nice! |
1b3c430
to
d189e91
Compare
This comment has been minimized.
This comment has been minimized.
extensions/vertx-http/dev-ui-resources/src/main/resources/dev-ui/qwc/qwc-theme-switch.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Phillip Kruger <phillip.kruger@gmail.com>
d189e91
to
7d1b7eb
Compare
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.
Nice! Sorry for bothering with the mousewheel, I personally find this quite annoying :D
Status for workflow
|
Yeah, I must admit I prefer it without the mousewheel behavior too. |
Fix #42297
A vaadin-web-components upgrade broke the theme switch. However, the theme switch was always a bit of a hack, example, you can click on the icon then all the options appear, and if you do not select one, but click off screen, all options disappear.
I change this to not use the menu item with the popup, but rather made the button just loop through all options [Desktop, Light, Dark] as you click. You can also use your mouse wheel to go up and down thought the themes.
I also moved the theme logic to it's own component, out of header. This is more maintainable.
/cc @xstefank @melloware