D22856: Enable do not disturb mode when screens are mirrored

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Jul 31 15:55:19 BST 2019


broulik created this revision.
broulik added reviewers: Plasma, VDG, dfaure.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  When two screens overlap during e.g. a presentation, automatically enable do not disturb mode.
  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.

TEST PLAN
  - Hit Meta+P, chose "mirror", dnd was engaged, popups hid
  - Hit Meta+P again, chose "extend left", dnd was disabled
  - Changed setting, it enabled/disabled live accordingly
  
  F7126785: Screenshot_20190731_164642.png <https://phabricator.kde.org/F7126785>
  
  Two questions:
  
  - 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
  - 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.
  
  There will be a followup patch to add a checkbox to notifications KCM

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  applets/notifications/package/contents/ui/FullRepresentation.qml
  applets/notifications/package/contents/ui/global/Globals.qml
  libnotificationmanager/CMakeLists.txt
  libnotificationmanager/kcfg/donotdisturbsettings.kcfg
  libnotificationmanager/mirroredscreenstracker.cpp
  libnotificationmanager/mirroredscreenstracker_p.h
  libnotificationmanager/settings.cpp
  libnotificationmanager/settings.h

To: broulik, #plasma, #vdg, dfaure
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190731/19d2dc4b/attachment.html>


More information about the Plasma-devel mailing list