<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG, dfaure.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22856">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When two screens overlap during e.g. a presentation, automatically enable do not disturb mode.<br />
As per usual with automatic do not disturb, it can be overruled by unchecking the checkbox in the notification plasmoid where it will be disabled until the screen configuration changes again and it still find a mirrored screen setup.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Hit Meta+P, chose "mirror", dnd was engaged, popups hid</li>
<li class="remarkup-list-item">Hit Meta+P again, chose "extend left", dnd was disabled</li>
<li class="remarkup-list-item">Changed setting, it enabled/disabled live accordingly</li>
</ul>

<p><a href="https://phabricator.kde.org/F7126785" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7126785: Screenshot_20190731_164642.png</a></p>

<p>Two questions:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Should this be enabled by default? Are there instances where you want to mirror but have notifications? Mind that you can still "one time" overrule this if you need them for once, until you change screen config again</li>
<li class="remarkup-list-item">Should the tooltip say anything, if so, what? We show "Do not disturb until $time" when a time is set but right now it doesn't say anything for mirrored screens. The applet icon changes to the crossed-out bell but the user has to click to find out what's going on.</li>
</ul>

<p>There will be a followup patch to add a checkbox to notifications KCM</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22856">https://phabricator.kde.org/D22856</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/package/contents/ui/FullRepresentation.qml<br />
applets/notifications/package/contents/ui/global/Globals.qml<br />
libnotificationmanager/CMakeLists.txt<br />
libnotificationmanager/kcfg/donotdisturbsettings.kcfg<br />
libnotificationmanager/mirroredscreenstracker.cpp<br />
libnotificationmanager/mirroredscreenstracker_p.h<br />
libnotificationmanager/settings.cpp<br />
libnotificationmanager/settings.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, dfaure<br /><strong>Cc: </strong>plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>