Bug 456321 - With certain Color Scheme settings, inactive window titlebar not colored with the setting turned on
Summary: With certain Color Scheme settings, inactive window titlebar not colored with...
Status: RESOLVED INTENTIONAL
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_colors (show other bugs)
Version: 5.25.3
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-04 13:12 UTC by linuxatemyram
Modified: 2022-10-01 20:25 UTC (History)
5 users (show)

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


Attachments
inactive titlebar color (1.78 MB, image/png)
2022-07-19 16:12 UTC, linuxatemyram
Details
Breeze Light color scheme with custom accent settings (3.47 KB, text/plain)
2022-07-19 17:57 UTC, linuxatemyram
Details
Custom-Colorscheme-Plasma-5.26.80 (3.44 KB, text/plain)
2022-09-26 19:44 UTC, Denys Madureira
Details
old-way-to-tint-inactive-titlebars (1.62 MB, image/png)
2022-09-27 16:56 UTC, Denys Madureira
Details
CustomColorSchemeWithTintTitlebar (2.93 MB, video/mp4)
2022-10-01 20:25 UTC, Denys Madureira
Details

Note You need to log in before you can comment on or make changes to this bug.
Description linuxatemyram 2022-07-04 13:12:05 UTC
SUMMARY
The setting "Apply accent color to inactive window titlebars" does not work. Inactive window titlebar will not get a custom color when the setting is turned on.


STEPS TO REPRODUCE
1. Choose a custom accent color, create a new color scheme from Breeze Light, go to color scheme settings and tick both header accent boxes, save theme and apply.

OBSERVED RESULT
Inactive window titlebar not getting a custom color.

EXPECTED RESULT
Inactive window titlebar getting a custom color.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20220703
KDE Plasma Version: 5.25.2
KDE Frameworks Version: 5.95.0
Qt Version: 5.15.5
Kernel Version: 5.18.6-1-default (64-bit)
Graphics Platform: Wayland

ADDITIONAL INFORMATION
This option worked on previous version of 5.25. Also I double checked the issue with KDE Neon on VM, it's the same there.
Comment 1 Nate Graham 2022-07-05 18:01:19 UTC

*** This bug has been marked as a duplicate of bug 455395 ***
Comment 2 linuxatemyram 2022-07-16 17:30:50 UTC
So this is not resolved/fixed in 5.25.3. Inactive titlebar is still not colored when the corresponding option is checked. 

This was marked as dublicate of https://bugs.kde.org/show_bug.cgi?id=455395 whichs says it's fixed in 5.25.3. So is this a separate issue or am I missing something?
Comment 3 linuxatemyram 2022-07-19 06:33:19 UTC
I'm reopening this as it's not fixed in Plasma 5.25.3. Repro instructions in original bug report.
Comment 4 Nate Graham 2022-07-19 15:34:00 UTC
Can you attach a screenshot that shows an active window and an inactive window, with both using your custom color scheme?
Comment 5 linuxatemyram 2022-07-19 16:12:03 UTC
Created attachment 150743 [details]
inactive titlebar color
Comment 6 linuxatemyram 2022-07-19 16:15:35 UTC
Ive attached it above. The color scheme is copy of Breeze Light, I renamed it Breeze Light Alt, applied the color settings to it and saved, then hit apply.
Comment 7 linuxatemyram 2022-07-19 16:38:09 UTC
And if I uncheck the inactive titlebar color setting, save and apply, the tiltebar color look exactly the same, which is why I think the feature is not workin?

https://imgur.com/JuzKiu0
Comment 8 Nate Graham 2022-07-19 16:57:07 UTC
Thanks. Cannot reproduce in current git master. Can you attach your custom color scheme file?
Comment 9 linuxatemyram 2022-07-19 17:57:41 UTC
Created attachment 150747 [details]
Breeze Light color scheme with custom accent settings
Comment 10 Nate Graham 2022-07-19 19:51:42 UTC
Can reproduce with that color scheme file.
Comment 11 Nate Graham 2022-07-19 19:55:19 UTC
This appears to be a config migration/sync issue. I can make your color scheme work for me by applying the following changes:

nate@Liberator:~/Desktop$  diff OLD.colors NEW.colors 
134,138c134,136
< TintFactor=0.15
< accentActiveTitlebar=true
< accentInactiveTitlebar=true
---
> TitlebarIsAccentColored=true

So this is already fixed in git master, but 5.25.x is broken, because we have a mismatch between the config data name and what the UI shows.
Comment 12 linuxatemyram 2022-07-19 20:36:29 UTC
Thank you for taking this under scrutiny!
Comment 13 Denys Madureira 2022-09-23 20:04:37 UTC
(In reply to Nate Graham from comment #11)
> This appears to be a config migration/sync issue. I can make your color
> scheme work for me by applying the following changes:
> 
> nate@Liberator:~/Desktop$  diff OLD.colors NEW.colors 
> 134,138c134,136
> < TintFactor=0.15
> < accentActiveTitlebar=true
> < accentInactiveTitlebar=true
> ---
> > TitlebarIsAccentColored=true
> 
> So this is already fixed in git master, but 5.25.x is broken, because we
> have a mismatch between the config data name and what the UI shows.

This bug is still present on Plasma 5.25.5. On test images of KDE Neon, with Plasma 5.26 beta, the option to tint inactive headers doesn't exist anymore. Won't we have this option anymore?
Comment 14 Nate Graham 2022-09-23 20:06:40 UTC
It was removed because now we always tint inactive headers when using this setting.
Comment 15 Denys Madureira 2022-09-24 16:25:30 UTC
(In reply to Nate Graham from comment #14)
> It was removed because now we always tint inactive headers when using this
> setting.

So, it should be working here, right?
https://i.imgur.com/pyHGnq3.png
https://i.imgur.com/A7eWDNx.png
But it is not applying to Dolphin, in this example, a inactive app.
I tested with the neon-unstable-20220922-0959 image.
Comment 16 Nate Graham 2022-09-26 17:20:49 UTC
Can you please attach your custom .colors file that I can use to reproduce the issue?
Comment 17 Denys Madureira 2022-09-26 19:44:18 UTC
Created attachment 152444 [details]
Custom-Colorscheme-Plasma-5.26.80

This is the color scheme. I just selected Breeze, tick the checkbox to tint headers and the window and increase a little the tint strength.
Comment 18 Denys Madureira 2022-09-26 19:45:01 UTC
(In reply to Nate Graham from comment #16)
> Can you please attach your custom .colors file that I can use to reproduce
> the issue?

Done.
Comment 19 Nate Graham 2022-09-27 14:53:21 UTC
Thanks. Seems to me that it's working; the inactive titlebar gets tinted with the accent color just like you told it to. However because you also told it to tint all other colors with the accent color too, this becomes no longer noticeable in comparison to the other colors. But it's still getting tinted.
Comment 20 Denys Madureira 2022-09-27 16:55:46 UTC
(In reply to Nate Graham from comment #19)
> Thanks. Seems to me that it's working; the inactive titlebar gets tinted
> with the accent color just like you told it to. However because you also
> told it to tint all other colors with the accent color too, this becomes no
> longer noticeable in comparison to the other colors. But it's still getting
> tinted.

Oh, so there is no more the option to the inactive titlebar gets tinted exactly as the active titlebar? I mean, with the strong color.
I will attach a screenshot on how it used to be.
Comment 21 Denys Madureira 2022-09-27 16:56:46 UTC
Created attachment 152468 [details]
old-way-to-tint-inactive-titlebars

So, there is no way to get this behavior again, right?
Comment 22 Nate Graham 2022-09-27 17:01:43 UTC
That's correct. This was done intentionally as a way of making it harder for people to accidentally create color schemes with no visual contrast between active and inactive windows.
Comment 23 Denys Madureira 2022-10-01 20:25:37 UTC
Created attachment 152542 [details]
CustomColorSchemeWithTintTitlebar

Ok, I'm here again. Sorry for that.

I want to know if this is the desired behavior:
The default Breeze Light and Dark have this behavior that we see in the beginning of the video - the titlebar of the inactive widow has a different contrast to differentiate.
So, when we use the option to tint the header based on these two color scheme, this effect happens. Ok. It's expected.
But, if we create a custom color scheme with a custom header (that doesn't change the contrast from inactive windows - as you also see in the video) and them enable the option to tint the titlebar on this color scheme, it change it behavior. Without this option the contrast don't change so, I was hoping that, after enable this option to this specific color scheme, the contrast doesn't change too.

Is this the expected behavior? To always do this, independent of the color scheme, of the user option and everything else? When you active the option to tint the titlebar it will always change the contrast of the inactive window and there is nothing that we can do?