luisbocanegra / plasma-panel-colorizer

Fully-featured widget to bring Latte-Dock and WM status bar customization features to the default KDE Plasma panel
https://store.kde.org/p/2130967
GNU General Public License v3.0
109 stars 0 forks source link

[Bug] Light/Dark theme transitions broken by Panel Colorizer #41

Closed marcosilva85 closed 1 month ago

marcosilva85 commented 2 months ago

Hi, I love the different ways Panel Colorizer enables to transform our setups. In my case, I use it simply to add more panel transparency and making a highlight on the Window Title widget.

Though it works, there is a small bug that is an annoyance when switching within light/dark themes, or by doing so via the kde Night-light feature that does this transition automatically on certain times.

Panel Colorizer messes up with the transition switch from dark->light or light->dark theme , and as a result the colors of the icons are kept in an incorrect state. This is a bug because I don't have any overrides on "Text and Icons", which is actually disabled. See screenshots attached - you can see because Panel Colorizer is active, sadly the icons and most of other widgets were not allowed to change out of the white color to black, when actually under a Light theme they should turn black: 1 dark 2 dark-to-light-problems See this last screenshot showing how a proper Breeze Light theme: 3 how-light-should-look-like

Interestingly - if I change any setting at all in Panel Colorizer and apply, suddenly the light/dark theme properly applies. Even if I toggle a setting and apply, like "contrast correction" from a section that I don't even have enabled (Text and Icons), still the theme suddenly properly applies, which is even more interesting.

I can send my profile if required, but should be easy to reproduce. Given that, perhaps there's an easy fix for this?

luisbocanegra commented 2 months ago

Can reproduce, will try to have it fixed as soon as I can. In the meantime you can enable Text and icons > Colors with these settings as a workaround:

Source: System Color: Text Color set: Background Contrast correction: unchecked

luisbocanegra commented 2 months ago

Interestingly - if I change any setting at all in Panel Colorizer and apply, suddenly the light/dark theme properly applies. Even if I toggle a setting and apply, like "contrast correction" from a section that I don't even have enabled (Text and Icons), still the theme suddenly properly applies, which is even more interesting.

Property applying from disabled sections should not change panel appearance, that is a bug.

Panel reloading will still happen when changing settings from disabled sections, because I reload the theme if any setting changes instead of having to check each of options (89 currently!), not a bug.

The reload is what is causing the colors to update, but if they are disabled they shouldn't change at all but follow the system instead, this is the bug causing the broken dark/light transition (probably just need to do something similar to what I did for https://github.com/luisbocanegra/plasma-wallpaper-effects/commit/aa3de2f6ebb9539a54708dea78e26a1b50a6d401)

luisbocanegra commented 1 month ago

Commit https://github.com/luisbocanegra/plasma-panel-colorizer/commit/c073e3e9c7b52b35053a9ad738956876ee0f93b5 should fix this.

To test do one of these:

plasmoid-panel-colorizer-v0.5.2-1.zip

luisbocanegra commented 1 month ago

Just noticed some errors, hold on the testing

luisbocanegra commented 1 month ago

I have updated the file, should be good now

luisbocanegra commented 1 month ago

This should be fixed in v0.5.2