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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-workspace/996f5b199f40a
                   |                            |de64b4d833f5ece5300c582670c
         Resolution|---                         |FIXED

--- Comment #10 from Marco Martin <notm...@gmail.com> ---
Git commit 996f5b199f40ade64b4d833f5ece5300c582670c by Marco Martin.
Committed on 27/01/2017 at 15:43.
Pushed by mart into branch 'master'.

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/996f5b199f40ade64b4d833f5ece5300c582670c

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

Reply via email to