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.
Created attachment 85181 [details] Added a apply button. I am just a beginner, so kindly excuse me if its completely wrong.
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.
Created attachment 85229 [details] Added a apply button. Added the apply button Handled the apply button setEnable() cases.
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.
The new config dialog in the upcoming 5.2.0 release has an apply button.