Bug 455395 - Tinted color is not applied to window titles if the color scheme doesn't define header colors
Summary: Tinted color is not applied to window titles if the color scheme doesn't defi...
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Unclassified
Component: kcm_colors (show other bugs)
Version: 5.25.0
Platform: Other Linux
: NOR normal (vote)
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-16 11:05 UTC by popov895
Modified: 2022-06-22 19:47 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.25.2


Attachments
Window title without tinted color (210.56 KB, image/png)
2022-06-16 11:05 UTC, popov895
Details
Color scheme without header and with tinted color feature (2.70 KB, text/plain)
2022-06-16 11:06 UTC, popov895
Details
kdeglobals (4.18 KB, text/plain)
2022-06-16 11:07 UTC, popov895
Details

Note You need to log in before you can comment on or make changes to this bug.
Description popov895 2022-06-16 11:05:50 UTC
Created attachment 149782 [details]
Window title without tinted color

SUMMARY

If the color scheme doesn't define header colors and uses the tinted color feature,, then the window title isn't tinted.

STEPS TO REPRODUCE

1. Apply a color scheme that doesn't define header colors, but uses the tinted color feature
2. Apply any accent color

OBSERVED RESULT

Window title isn't tinted.

EXPECTED RESULT

Window title is tinted.

SOFTWARE/OS VERSIONS

Operating System: openSUSE Tumbleweed 20220613
KDE Plasma Version: 5.25.0
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.2
Graphics Platform: X11
Comment 1 popov895 2022-06-16 11:06:47 UTC
Created attachment 149783 [details]
Color scheme without header and with tinted color feature
Comment 2 popov895 2022-06-16 11:07:40 UTC
Created attachment 149784 [details]
kdeglobals
Comment 3 Nate Graham 2022-06-16 19:51:15 UTC
Will be fixed by https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1844.
Comment 4 Nate Graham 2022-06-17 20:12:18 UTC
Fixed by Jan Blackquill with https://invent.kde.org/plasma/plasma-workspace/-/commit/a671178a8f35b87d385bce9cfcf72603da25f586 in Plasma 5.25.1!
Comment 5 popov895 2022-06-21 17:20:16 UTC
Still not fixed.
Comment 6 Bug Janitor Service 2022-06-22 09:15:43 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1865
Comment 7 Nate Graham 2022-06-22 19:27:08 UTC
Git commit 2d272745eb0ec6da96b45ea66842cba7a3e69f21 by Nate Graham, on behalf of Eugene Popov.
Committed on 22/06/2022 at 19:27.
Pushed by ngraham into branch 'master'.

kcms/colors: Fix window titlebar tinting in colorsapplicator

Tint the window titlebars even if the option to make them accent-colored is not
explicitly enabled; in this case you want them tinted anyway because they're a
part of the fill window which will get tinted.
FIXED-IN: 5.25.2

M  +4    -6    kcms/colors/colorsapplicator.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/2d272745eb0ec6da96b45ea66842cba7a3e69f21
Comment 8 Nate Graham 2022-06-22 19:47:52 UTC
Git commit b6b34f14901d8e493be4caa7f54c2dd36a2f0b46 by Nate Graham, on behalf of Eugene Popov.
Committed on 22/06/2022 at 19:47.
Pushed by ngraham into branch 'Plasma/5.25'.

kcms/colors: Fix window titlebar tinting in colorsapplicator

Tint the window titlebars even if the option to make them accent-colored is not
explicitly enabled; in this case you want them tinted anyway because they're a
part of the fill window which will get tinted.
FIXED-IN: 5.25.2


(cherry picked from commit 2d272745eb0ec6da96b45ea66842cba7a3e69f21)

M  +4    -6    kcms/colors/colorsapplicator.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/b6b34f14901d8e493be4caa7f54c2dd36a2f0b46