Bug 426156

Summary: Checking and un-checking "Use theme's default window border size" checkbox leaves Apply button active
Product: [Applications] systemsettings Reporter: postix <postix>
Component: kcm_kwindecorationAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: cyril.rossi, mail, nate, plasma-bugs-null, postix
Priority: NOR    
Version First Reported In: 5.19.4   
Target Milestone: ---   
Platform: Other   
OS: Other   
Latest Commit: Version Fixed/Implemented In: 5.21
Sentry Crash Report:
Attachments: Screenshot
Screenshot

Description postix 2020-09-03 18:47:44 UTC 
STEPS TO REPRODUCE
1. Go to global design
2. Check "Default setting for the windows"
3. Uncheck it again

OBSERVED RESULT

Apply button stays active


EXPECTED RESULT

If nothing has changed, there's nothing to apply.


SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux
KDE Plasma Version: 5.19.4
KDE Frameworks Version: 5.73.0
Qt Version: 5.15.0
Comment 1 Nate Graham 2020-09-08 16:49:06 UTC 
Which KCM exactly? The Window Decorations KCM?
Comment 2 postix 2020-09-08 19:38:42 UTC 
Created attachment 131492 [details]
Screenshot

This one. :)
Comment 3 Nate Graham 2020-09-08 20:04:55 UTC 
Ah! Same as Bug 426155 then, no?
Comment 4 postix 2020-09-08 22:03:41 UTC 
Created attachment 131498 [details]
Screenshot

(In reply to Nate Graham from comment #3)
> Ah! Same as Bug 426155 then, no?

Sorry, I confused it. This one. The other one was for 426155. :)
Comment 5 Nate Graham 2020-09-09 19:17:28 UTC 
Ah yes, so the window decoration KCM. :)
Comment 6 Cyril Rossi 2020-10-14 13:06:09 UTC 
If we talk about the checkbox "Use theme's default border size" on KCM KWin Decoration, I don't reproduce it on Plasma 5.20.80

It should have been fixed by the work for the highlight changed.
Comment 7 postix 2020-10-14 13:43:32 UTC 
(In reply to Cyril Rossi from comment #6)
> If we talk about the checkbox "Use theme's default border size" on KCM KWin
> Decoration, I don't reproduce it on Plasma 5.20.80

Yes, we do.

> It should have been fixed by the work for the highlight changed.
That's good news! I'm still on 5.19.5, but if Nate cannot reproduce it any longer as well, feel free to mark it as "resolved fixed". :)
Comment 8 Nate Graham 2020-10-14 14:47:31 UTC 
Can confirm that this is fixed now. :)