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