This patch moves the Desktop Effects KCM out of the Workspace Behavior category and
into the top-level Appearance section, as discussed in T8871.
Details
Details
- Reviewers
- None
- Group Reviewers
VDG Plasma - Maniphest Tasks
- T11314: Systematic KCM reorganisation: Appearance section
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Branch
- relocate-desktop-effects (branched from master)
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 7356 Build 7374: arc lint + arc unit
Comment Actions
On hold until all three are accepted, and there's some unresolved discussion in D18419.
Comment Actions
The goal of moving effects which are only related to eye-candy to Appearance definitely makes sense and is a goal I 1000% support.
But the current KCM is a mixture of high level selection a user should be meddling with and low level tech things which happen to use the effects plugin structure.
Things like blur settings are purely tech focussed, zoom is purely behvaioural accessibility and "showFPS" shouldn't exist in the UI.
I think it's a good example of a case where we need to enforce ground-up rethinking and moving just the releavnt parts.
We can make a kwin task about splitting this.
Comment Actions
It misleadingly includes the word "Desktop", which in the typical user's understanding for this context, is the thing with the wallpaper that has files and folders and icons on it.
Comment Actions
People have been using "desktop effects" term since Compiz was all the rage and perhaps even before that. So, I doubt the "desktop" part causes confusion.
Comment Actions
Regular users don't have any idea what Compiz is (or was). Our user interfaces are designed for normal people, not technical experts.
Anyway, we can continue the discussion in D22894 now that I've submitted it. I wound encourage KWin folks to give the VDG the benefit of the doubt when it comes to the user interface though, the way we have been encouraged to give KWin developers the benefit of the doubt regarding implementation details.