D29130: [Applet] Show in system tray iff at least one vault is open

[Konstantinos Smanis]

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

REVISION SUMMARY
  Currently, the "Shown when relevant" visibility option in the system
  tray shows an icon iff at least one vault exists, regardless of open
  status (i.e., if it is open or closed). Change this behavior to show an
  icon iff at least one *open* vault exists, allowing the user to be more
  mindful of potentially sensitive information being accessible.
  
  The previous behavior can be emulated by selecting the "Always shown"
  visibility option, assuming that the user does not create and delete
  Plasma Vaults for recreation.

TEST PLAN
  Compiles and runs as expected both on master (commit 4d11dcf <https://phabricator.kde.org/R845:4d11dcffd65c5b98e10678fe904baa789f89bc2f>) and
  Plasma/5.18 (commit 1f546c1 <https://phabricator.kde.org/R845:1f546c1d6c3f20fcab8ee4b9d2d4090ecf17dc75>).

REPOSITORY
  R845 Plasma Vault

BRANCH
  applet-systray-relevance (branched from master)

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

AFFECTED FILES
  plasma/package/contents/ui/main.qml
  plasma/vaultsmodel.cpp
  plasma/vaultsmodel.h
  plasma/vaultsmodel_p.h

To: ksmanis, #plasma, #vdg
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, 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/20200423/251e0d5e/attachment.html>