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

Authored by broulik on Oct 15 2019, 7:33 PM.

Details

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

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Oct 15 2019, 7:33 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 15 2019, 7:33 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Oct 15 2019, 7:33 PM
ngraham accepted this revision.Oct 15 2019, 10:26 PM
This revision is now accepted and ready to land.Oct 15 2019, 10:26 PM
This revision was automatically updated to reflect the committed changes.