Bug 463702 - The accent color picked from wallpaper should take window background color into account
Summary: The accent color picked from wallpaper should take window background color in...
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_colors (show other bugs)
Version: 5.26.4
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2023-01-01 19:20 UTC by Akseli Lahtinen
Modified: 2024-09-17 09:36 UTC (History)
5 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Akseli Lahtinen 2023-01-01 19:20:02 UTC
SUMMARY

This is more an accessibility thing, but when using picking accent color from wallpaper setting, the accent color should also take the window background color into account. Like mentioned here, there should be 4.5:1 ratio between foreground and background color for accessibility reasons: https://randoma11y.com/

Of course, there will always be color combinations that just do not work. But keeping good contrast between accent and background color should help a lot. At the very least, it would keep items readable even if the accent color is not desirable.

https://randoma11y.com/?color=%23202020 For example, this gray is my background color. The color picking method should use that background color and calculate a proper accent color that keeps a good contrast between background and foreground.

No idea if something like this is already in the works so here's my system information.

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 37
KDE Plasma Version: 5.26.4
KDE Frameworks Version: 5.101.0
Qt Version: 5.15.7
Kernel Version: 6.0.15-300.fc37.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 3600 6-Core Processor
Memory: 15,5 GiB of RAM
Graphics Processor: AMD Radeon RX 6600

I could also look into this. :)
Comment 1 Nate Graham 2023-01-06 21:40:45 UTC
We used to, but people complained a lot about it. See Bug 442820. However it's possible the implementation was to blame, rather than the concept per se. We can revisit it.
Comment 2 Akseli Lahtinen 2024-09-17 09:36:09 UTC
This was fixed with https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4370