D24679: [Notifications] Also consider enabled and connected for screen to be checked against

[Kai Uwe Broulik]

broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  Fixes erroneously enabling do not disturb mode in some situations when having screens mirrored and then closing the lid. You'll have two overlapping screens but one of them is disabled so should be ignored.
  Depending on the order of outputs this may or may not happen (which is why I didn't notice earlier when I tested with my new laptop).

TEST PLAN
  5.17 branch
  
  - Didn't actually test it yet but I'll give it some testing the other day

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D24679

AFFECTED FILES
  libnotificationmanager/mirroredscreenstracker.cpp

To: broulik, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191015/e666c8d9/attachment-0001.html>