SystemTray: QML version #2

[Dmitry Ashkadov]

15.10.2012 20:20, Aaron J. Seigo ?????:
> On Monday, October 15, 2012 19:20:13 Dmitry Ashkadov wrote:
>
> i do notice some small visual issues in the screenshots (thanks for providing
> those, btw!), mostly to do with spacing and margins in the popup. as you can
> see in image 05, there is not enough margin around the item and so the
> background item appears too close and even touches the icon. similarly, there
> should be a bit more space between the icon and the text imo (0.5 or even 1
> em?)
>
I'll compare it to C++ applet and try increase margins.

>> case of C++ system tray applet. I can see many messages:
>>> .../apps/plasma/plasmoids/battery/contents/ui/batterymonitor.qml:183:
>>> TypeError: Result of expression 'pmSource.data["PowerDevil"]'
> hm.. and if you put the battery widget directly into the panel, or on the
> desktop or dashboard, it works?
Battery works if I click on its icon, but if I click on name (text) then 
battery doesn't work. I think it is a regression, because I've found out 
several facts:

  * Old C++ tray has the same problem (git checkout master to switch to
    C++ tray)
  * Battery applet doesn't work but applet for Network manager works if
    I click on its name
  * KDE 4.8 is now system KDE and battery works as expected, but in
    locally installed KDE (master or 4.9) battery doesn't work, it
    doesn't work on KDE 4.9 -- future release of my distro.


Thank you!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121015/ca8b2439/attachment.html>