From kde-core-devel Mon Jan 06 11:14:22 2014 From: =?ISO-8859-1?Q?Aur=E9lien_G=E2teau?= Date: Mon, 06 Jan 2014 11:14:22 +0000 To: kde-core-devel Subject: Re: KClasses vs. Qt5Classes Message-Id: <2431254.9yB7i3KWtd () trinity> X-MARC-Message: https://marc.info/?l=kde-core-devel&m=138900689911541 Le mardi 31 d=E9cembre 2013 12:42:22 Martin Graesslin a =E9crit : > On Tuesday 31 December 2013 12:15:09 David Faure wrote: > > > QSystemTrayIcon =3D> KNotificationItem > >=20 > > No clue. I can't even find KNotificationItem in KF5 anywhere.... !?= !? > > In fact it doesn't exist in kdelibs4 either. > >=20 > > I think it got replaced with KStatusNotifierItem since 4.4 ? > > That one is still valid in KF5 (framework "knotifications"). > > I have no idea if/why it means QSystemTrayIcon is bad though. >=20 > QSystemTrayIcon uses XEmbed which is bad by definition ;-) >=20 > We have discussed this in the plasma team and think that the best sol= ution > would be to extend QSystemTrayIcon to use the status notifier API if > available. Might need some hooks into the QPA as we probably cannot d= epend > on D-Bus on that level. But as that doesn't exist yet, at the moment = the > proper suggestion should be to use KStatusNotifierItem. For what it's worth, I created something like that for Qt4 while I was = at=20 Canonical: sni-qt [1]. It requires a Qt patch though. The patch is in the sni-qt tarball (and = here=20 [2]) Aur=E9lien [1]: https://launchpad.net/sni-qt [2]: http://bazaar.launchpad.net/~indicator-applet-developers/sni-qt/tr= unk.13.04/view/head:/patches/qsystemtrayicon-plugin-system-4.7.4.diff