mart created this revision. mart added reviewers: Plasma, sebas. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION 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 BUG:375507 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 REPOSITORY R120 Plasma Workspace BRANCH phab/redundant REVISION DETAIL https://phabricator.kde.org/D4309 AFFECTED FILES shell/shellcorona.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart, #plasma, sebas Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
