Bug 402701

Summary:	"Desktop effects" filtering checkboxes don't get checked
Product:	[Applications] systemsettings	Reporter:	alt.su-7oon2u5b
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nate
Priority:	NOR
Version:	5.14.4
Target Milestone:	---
Platform:	Neon
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=396725
Latest Commit:	https://commits.kde.org/kwin/d28827626931ed9f8fbfd618bbdcdf3fdce6a5e6	Version Fixed In:	5.15.0

Description alt.su-7oon2u5b 2018-12-30 08:57:05 UTC

SUMMARY


STEPS TO REPRODUCE
1. Open System Settings > Desktop Behavior > Desktop Effects
2. Open the filtering dropdown menu (between the search bar and "Get New Effects")
3. Attempt to click / "check" the checkboxes

OBSERVED RESULT
The checkboxes don't become checked, but their background color changes to blue (default Breeze theme). The filtering operation indicated by their description seems to apply. 


EXPECTED RESULT
The checkboxes should become checked when clicked / "checked"

Comment 1 Nate Graham 2019-01-01 01:08:26 UTC

They do actually get checked, but so does the background. Which is weird.

Comment 2 Nate Graham 2019-01-01 17:09:28 UTC

Porting the KCM to QQC2 should fix this issue, as well as Bug 396725. I'm working on this.

Comment 3 Nate Graham 2019-01-02 19:45:00 UTC

Here's the patch: https://phabricator.kde.org/D17928

Comment 4 Nate Graham 2019-01-06 23:09:13 UTC

Git commit d28827626931ed9f8fbfd618bbdcdf3fdce6a5e6 by Nate Graham.
Committed on 06/01/2019 at 23:09.
Pushed by ngraham into branch 'master'.

[kcmkwin/compositing] Port Effects KCM to QQC2

Summary:
This patch ports the Effects KCM to QQC2, which yields the following benefits:
1. General performance enhancements of only using QQC2 rather than a mix of 1 and 2
2. Some code simplification is possible
3. Improves the appearance of the checkable menu items in the dropdown menu, fixing https://bugs.kde.org/show_bug.cgi?id=402701
4. Improves the appearance when using a fractional scale factor, fixing https://bugs.kde.org/show_bug.cgi?id=396725
5. Fixes the incorrect size when opened from `kcmshell5`

Along the way, two visual changes are introduced as a by-product of porting:
1. The scrollbar is inline, so it overlaps some of the buttons in the content. This is somewhat undesirable, but adopting a Kirigami scrollview would fix this.
2. The button that displays a dropdown menu no longer has a downward-pointing arrow to indicate as such. This is not my preference, but there's a benefit to being consistent, and eventually we could can change this in one place to impeove the appearance of buttons that display dropdown menus everywhere in one fell swoop.
Related: bug 396725, bug 396076
FIXED-IN: 5.15.0

Test Plan:
All functionality still works

In System Settings, showing improved checkable menu item appearance:
{F6523385}

In `kcmshell` at 1x scale:
{F6523386}

In `kcmshell` at 1.5x scale:
{F6523387}

Reviewers: #kwin, #plasma, davidedmundson

Reviewed By: #kwin, #plasma, davidedmundson

Subscribers: davidedmundson, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D17928

M +60 -63 kcmkwin/kwincompositing/qml/EffectView.qml

https://commits.kde.org/kwin/d28827626931ed9f8fbfd618bbdcdf3fdce6a5e6