Diffusion KWin 746fc49bb6a3

Revert "desktop grid: zoom hovered window"

Authored by sebas on Mar 24 2016, 1:42 PM.

Description

Revert "desktop grid: zoom hovered window"

Summary:
This reverts commit 14d1fdf63fc27d908f062695e85fa55dde725b55.

The zoom effect is definitely the wrong way, it's broken (can be
triggered without moving the mouse, expands windows outside of their
allotted area (looks like broken sizing/placement), isn't animated so
feels very choppy, isn't using well-known highlight semantics), the
result is that it feels unnatural and jarring. To be honest, when I
looked at it, I was looking for a bug in the code, e.g. margins not
being taken into account. It didn't come up in me that this could be
wanted behavior, until I read the code.

As to the original commit message: The highlight is useless for this
case, as any window can be dragged or activated, highlighted or not.

What *could* work is to somehow intensify the colors, but that again
would have to be animated, and play well with the ongoing desktop-zoom
animation, the mouse pointer location changes relative to the window, so
it's really complex to get right. I thought of this for a while, I don't
think the complexity that has to be implemented is worth the benefit,
because, what does highlighted really mean here? "window is under the
mouse" isn't a useful metric, as we don't know if the user is changing
desktops or rearranging windows, and "window under mouse" changes
depending on the zoom level, "window can be dragged or activated" also
isn't a useful metric, as I can do that with any window, anyway.

Test Plan: Tested with patch applied, desktopgrid windows don't jump around anymore.

Reviewers: graesslin

Reviewed By: graesslin

Subscribers: plasma-devel

Projects: Plasma

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

Details

Committed
sebasMar 24 2016, 1:52 PM
Reviewer
graesslin
Differential Revision
D1209: Revert "desktop grid: zoom hovered window"
Parents
R108:e62dad48a67f: Add a surface tree to DebugConsole
Branches
Unknown
Tags
Unknown
Reverts
R108:14d1fdf63fc2: desktop grid: zoom hovered window