Bug 442478 - Breeze Light color scheme: Selection Active Text is blue when it should be white
Summary: Breeze Light color scheme: Selection Active Text is blue when it should be white
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Color scheme (show other bugs)
Version: 5.22.4
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-15 14:47 UTC by Oleg Solovyov
Modified: 2021-09-17 20:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.23
Sentry Crash Report:


Attachments
invisible text (16.13 KB, image/jpeg)
2021-09-15 14:47 UTC, Oleg Solovyov
Details
Breeze theme (18.79 KB, image/jpeg)
2021-09-17 06:59 UTC, Oleg Solovyov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Solovyov 2021-09-15 14:47:09 UTC
Created attachment 141561 [details]
invisible text

SUMMARY
Using Breeze Light color theme, some texts become invisible.
e.g. profile list in ktouch. Selected profile is unreadable because text and background colors are equal

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 5.11.0
KDE Plasma Version: 5.22.5
KDE Frameworks Version: 5.86.0
Qt Version: 5.15.3
Comment 1 Nate Graham 2021-09-16 17:53:11 UTC
The colors are correct in the color scheme; this is an app bug where it inappropriately hardcodes some colors.
Comment 2 Oleg Solovyov 2021-09-17 06:59:06 UTC
Created attachment 141639 [details]
Breeze theme

(In reply to Nate Graham from comment #1)
> The colors are correct in the color scheme; this is an app bug where it
> inappropriately hardcodes some colors.

I don't think so.
Same app, same versions, different color scheme (Breeze vs. Breeze Light)
Comment 3 Nate Graham 2021-09-17 14:59:08 UTC
Hmm yeah looks like Selection Active Text is wrong in Breeze Light. It's currently blue when it should be white.

However in general the code in question is crying out for using Kirigami.BasicListItem instead, which would probably save hundreds of lines of code and fix bugs like this.
Comment 4 Nate Graham 2021-09-17 19:55:05 UTC
Git commit 09a0b5199c9653a4aa357506b8cb5aac7999f369 by Nate Graham.
Committed on 17/09/2021 at 19:53.
Pushed by ngraham into branch 'master'.

Fix incorrect active selection color in Breeze Light color scheme

Looks like an oversight at some point in the last, as all three other
Breeze* color schemes have the correct value here.
FIXED-IN: 5.23

M  +1    -1    colors/BreezeLight.colors

https://invent.kde.org/plasma/breeze/commit/09a0b5199c9653a4aa357506b8cb5aac7999f369
Comment 5 Nate Graham 2021-09-17 19:55:35 UTC
Git commit 2051d29ba7232726c4e795d940dc1edde1a98ba1 by Nate Graham.
Committed on 17/09/2021 at 19:55.
Pushed by ngraham into branch 'Plasma/5.23'.

Fix incorrect active selection color in Breeze Light color scheme

Looks like an oversight at some point in the last, as all three other
Breeze* color schemes have the correct value here.
FIXED-IN: 5.23
(cherry picked from commit 09a0b5199c9653a4aa357506b8cb5aac7999f369)

M  +1    -1    colors/BreezeLight.colors

https://invent.kde.org/plasma/breeze/commit/2051d29ba7232726c4e795d940dc1edde1a98ba1
Comment 6 Nate Graham 2021-09-17 20:00:52 UTC
Note that because of the way colors are serialized to the ~./config/kdeglobals file, you will need to switch you color scheme to something else and then back to Breeze light to pick up the change. We ultimately want to make that not be required; see https://phabricator.kde.org/T13663.