Review Request 120243: Remove fallback QSystemTray support

David Edmundson david at davidedmundson.co.uk
Thu Oct 2 14:25:55 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120243/
-----------------------------------------------------------

(Updated Oct. 2, 2014, 2:25 p.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks.


Repository: knotifications


Description
-------

Remove fallback QSystemTray support

Currently when we make a StatusNotifierItem if it fails we try to create
a classic X system tray icon via Qt's QSystemTray.

QSystemTray now (5.4) has a QPT backend that uses KStatusNotifierItem. This
means it's possible to infinite loop.

This patch removes the legacy QSystemTray support to avoid that
happening.

Downside: it means if a frameworks app is run on a platform without statusnotifier support but still has legacy X system tray support we would now gets no icons, but that would break on Wayland anyway.


Diffs
-----

  src/kstatusnotifieritem.h 7da1451 
  src/kstatusnotifieritem.cpp 767537d 
  src/kstatusnotifieritemprivate_p.h 051cc38 

Diff: https://git.reviewboard.kde.org/r/120243/diff/


Testing
-------

Ran statusnotifiertest (the one in plasma-workspace) with plasma running. Everything is fine.
Ran without plasma running and nothing happened (without kded5 infinite looping)


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141002/27c94504/attachment.html>


More information about the Kde-frameworks-devel mailing list