Bug 311841

Summary: [JJ] No apply button in systemsettings/workspace appearance/window decorations/configure decoration
Product: [Plasma] kwin Reporter: Jussi Schultink <jussi01>
Component: decorationsAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist CC: avinashzebisky
Priority: NOR Keywords: junior-jobs
Version: 4.9.90 (Beta 2)   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=325946
Latest Commit: Version Fixed In: 5.2.0
Sentry Crash Report:
Bug Depends on:    
Bug Blocks: 303462    
Attachments: Added a apply button.
Added a apply button.

Description Jussi Schultink 2012-12-17 18:04:25 UTC 
As title says, should be apply not ok in that dialog (or maybe apply and ok?) if you follow the same as the systemsettings style. 

Platform Version 4.9.90

Reproducible: Always

Steps to Reproduce:
1.Open up systemsettings/workspace appearance/window decorations/configure decoration
2. Change something
3. Look for the change and don't see it
4. Click Ok, the change appears.
Comment 1 Swarn Avinash Kumar 2014-02-16 16:58:56 UTC 
Created attachment 85181 [details]
Added a apply button.

I am just a beginner, so kindly excuse me if its completely wrong.
Comment 2 Martin Flöser 2014-02-17 07:06:32 UTC 
please use git.reviewboard.kde.org for patch suggestion with review group KWin. Here on the bugtracker it's very likely that we just forget about the patch.
Comment 3 Swarn Avinash Kumar 2014-02-19 06:10:26 UTC 
Created attachment 85229 [details]
Added a apply button.

Added the apply button
Handled the apply button setEnable() cases.
Comment 4 Swarn Avinash Kumar 2014-02-19 06:17:11 UTC 
Actually, I tried to it but it says - 

The file 'kwin/kcmkwin/kwindecoration/configdialog.cpp' (r5083d60) could not be found in the repository

I used ->
$ git diff HEAD~2 > final2.diff

I have little experience in git, so I couldn't figure out why?

(In reply to comment #2)
> please use git.reviewboard.kde.org for patch suggestion with review group
> KWin. Here on the bugtracker it's very likely that we just forget about the
> patch.
Comment 5 Martin Flöser 2015-01-05 14:11:41 UTC 
The new config dialog in the upcoming 5.2.0 release has an apply button.