D26447: Dynamically show and hide based on whether or not any vaults are configured

[Nathaniel Graham]

ngraham added a comment.


  In D26447#588486 <https://phabricator.kde.org/D26447#588486>, @ivan wrote:
  
  > For some reason, hiding this icon when there are no vaults was deemed undesired before. I don't recall why as I think it is a good idea. :)
  >
  > Maybe it was always shown for discoverability purposes... don't know.
  
  
  Probably that, though VDG discussion on the matter was strongly negative on the idea of putting more things in the system tray's visible section for discoverability purposes.
  
  > Is there a reason why this is a `count` instead of (or in addition to) just being `isEmpty` or something similar since the actual count is not used?
  
  It was a smaller code change than adding a new `isEmpty` property, and I suppose exposing `count` as a property could be useful in other contexts too.

REPOSITORY
  R845 Plasma Vault

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

To: ngraham, broulik, ivan, #plasma, #vdg
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, 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/20200106/fff9d965/attachment.html>