https://bugs.kde.org/show_bug.cgi?id=363804
Martin Gräßlin <mgraess...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |http://commits.kde.org/kwin
| |/e5fe3137b8a1b82360aa1507b6
| |bcee65089d93b1
Resolution|--- |FIXED
Status|CONFIRMED |RESOLVED
--- Comment #12 from Martin Gräßlin <mgraess...@kde.org> ---
Git commit e5fe3137b8a1b82360aa1507b6bcee65089d93b1 by Martin Gräßlin.
Committed on 15/06/2016 at 12:48.
Pushed by graesslin into branch 'master'.
Fix the ignore struts multi-screen handling
Summary:
The checks in Client::adjustedClientArea were a little bit too
agressive, excluding also valid setups.
This change addresses the regression by keeping the actual intended
improvements in place.
The check in Client::adjustedClientArea is now only done for the
special case of desktopArea == area. This ensures that a strut excluding
a complete screen won't affect the overall workarea.
In addition new checks are introduced in Workspace::updateClientArea.
When calculating the new sareas a check is performed whether the
intersection with the adjustedClientArea would result in the sarea
becoming empty (thus a screen being completely removed). If that's the
case the geometry is ignored to not exclude a complete screen.
Interestingly I should have noticed that something with the logic is
odd. As the test case had two commented geometries which we now get.
Reviewers: #plasma, apol, lbeltrame
Subscribers: plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D1744
M +2 -4 autotests/wayland/struts_test.cpp
M +19 -13 geometry.cpp
http://commits.kde.org/kwin/e5fe3137b8a1b82360aa1507b6bcee65089d93b1
--
You are receiving this mail because:
You are watching all bug changes.
