kwin/effects/desktopgrid
Sebastian Kügler 746fc49bb6 Revert "desktop grid: zoom hovered window"
Summary:
This reverts commit 14d1fdf63f.

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
2016-03-24 14:52:03 +01:00
..
CMakeLists.txt Use SERVICE_TYPES parameter to kcoreaddons_desktop_to_json() 2015-12-09 23:42:45 +00:00
desktopgrid.cpp Revert "desktop grid: zoom hovered window" 2016-03-24 14:52:03 +01:00
desktopgrid.h Revert "desktop grid: zoom hovered window" 2016-03-24 14:52:03 +01:00
desktopgrid.kcfg desktop grid: add option to hide buttons 2015-10-29 23:36:10 +01:00
desktopgrid_config.cpp [effects] Properly set the componentDisplayName on the KActionCollection 2015-08-12 09:40:55 +02:00
desktopgrid_config.desktop SVN_SILENT made messages (.desktop file) - always resolve ours 2016-01-30 09:40:55 +00:00
desktopgrid_config.h Use Q_SLOTS and Q_SIGNALS instead of slots and signals 2013-07-24 09:46:54 +02:00
desktopgrid_config.ui desktop grid: add option to hide buttons 2015-10-29 23:36:10 +01:00
desktopgridconfig.kcfgc Use KConfigXT in DesktopGrid effect 2012-09-15 11:24:11 +02:00
main.qml desktop grid: use -/+ text buttons 2015-10-29 23:36:10 +01:00