Bug 402701 - "Desktop effects" filtering checkboxes don't get checked
Summary: "Desktop effects" filtering checkboxes don't get checked
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: general (show other bugs)
Version: 5.14.4
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-12-30 08:57 UTC by alt.su-7oon2u5b
Modified: 2019-01-06 23:09 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.15.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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