D19745: Fix system tray UI/UX & refactor

[David Edmundson]

davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> ratijastk wrote in CompactApplet.qml:34
> As far as I remember, one of the branches never got executed in my environment. Could you point to specific use case which requires this "almost dead" code?

You will enter this branch if the system tray is on the left.

It doesn't make a difference now since Dialog gained support to not not overlap panels and to switch edges if it doesn't fit.

This puts the location in a "wrong" way we hit the conflict detection in Dialog and it fixes it.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  fix-system-tray

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

To: ratijastk, #vdg, #plasma, broulik, mart, hein, davidedmundson
Cc: davidre, davidedmundson, ngraham, ndavis, anthonyfieroni, plasma-devel, jraleigh, 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/20190518/eff59188/attachment.html>