https://bugs.kde.org/show_bug.cgi?id=375507

Marco Martin <notm...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://commits.kde.org/pla |https://commits.kde.org/pla
                   |sma-workspace/996f5b199f40a |sma-workspace/3792ef9e51dc0
                   |de64b4d833f5ece5300c582670c |ba21740e69964f26e241df7aed2

--- Comment #11 from Marco Martin <notm...@gmail.com> ---
Git commit 3792ef9e51dc0ba21740e69964f26e241df7aed2 by Marco Martin.
Committed on 27/01/2017 at 15:44.
Pushed by mart into branch 'Plasma/5.8'.

fix isOutputRedundant logic

Summary:
isOutputRedundant badly failed when two screens had
the exact same geometry (that should be, the base
case for "cloning" screens.
now the logic is:
a screen is redundant if:
* its geometry is contained in another one
* if their resolutions are different, the "biggest" one wins
* if they have the same geometry, the one with the lowest id wins (arbitrary,
but gives reproducible behavior and makes the primary

Test Plan:
tested with two screens:
* overlapping, same resolution: the lowest id (0, the primary) wins, the other
one doesn't get a view
* overlapping, different resolutions: the biggest one wins
* not overlapping: both get a view

Reviewers: sebas, #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: davidedmundson, plasma-devel

Tags: #plasma

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

M  +30   -5    shell/shellcorona.cpp

https://commits.kde.org/plasma-workspace/3792ef9e51dc0ba21740e69964f26e241df7aed2

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to