[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KClasses vs. Qt5Classes
From:       Aleix Pol <aleixpol () kde ! org>
Date:       2014-01-03 2:51:22
Message-ID: CACcA1Ro_yiv7QDbpAYcYqOp5=B6TuHVHM57fpBaGkicGH4y5uw () mail ! gmail ! com
[Download RAW message or body]

On Tue, Dec 31, 2013 at 12:42 PM, Martin Graesslin <mgraesslin@kde.org>wrote:

> On Tuesday 31 December 2013 12:15:09 David Faure wrote:
> > > QSystemTrayIcon => KNotificationItem
> >
> > No clue. I can't even find KNotificationItem in KF5 anywhere.... !?!?
> > In fact it doesn't exist in kdelibs4 either.
> >
> > 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.
> QSystemTrayIcon uses XEmbed which is bad by definition ;-)
>
> We have discussed this in the plasma team and think that the best solution
> would be to extend QSystemTrayIcon to use the status notifier API if
> available.
> Might need some hooks into the QPA as we probably cannot depend on D-Bus on
> that level. But as that doesn't exist yet, at the moment the proper
> suggestion
> should be to use KStatusNotifierItem.
>
> Cheers
> Martin


FWIW, our QPA (or QPT, actually) already depends on QtDBus.

Aleix

[Attachment #3 (text/html)]

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, \
Dec 31, 2013 at 12:42 PM, Martin Graesslin <span dir="ltr">&lt;<a \
href="mailto:mgraesslin@kde.org" \
target="_blank">mgraesslin@kde.org</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><div class="im">On Tuesday 31 December 2013 \
12:15:09 David Faure wrote:<br> &gt; &gt; QSystemTrayIcon =&gt; \
KNotificationItem<br> &gt;<br>
&gt; No clue. I can&#39;t even find KNotificationItem in KF5 anywhere.... \
!?!?<br> &gt; In fact it doesn&#39;t exist in kdelibs4 either.<br>
&gt;<br>
&gt; I think it got replaced with KStatusNotifierItem since 4.4 ?<br>
&gt; That one is still valid in KF5 (framework \
&quot;knotifications&quot;).<br> &gt; I have no idea if/why it means \
QSystemTrayIcon is bad though.<br> </div>QSystemTrayIcon uses XEmbed which \
is bad by definition ;-)<br> <br>
We have discussed this in the plasma team and think that the best \
solution<br> would be to extend QSystemTrayIcon to use the status notifier \
API if available.<br> Might need some hooks into the QPA as we probably \
cannot depend on D-Bus on<br> that level. But as that doesn&#39;t exist \
yet, at the moment the proper suggestion<br> should be to use \
KStatusNotifierItem.<br> <br>
Cheers<br>
<span class="HOEnZb"><font \
color="#888888">Martin</font></span></blockquote></div><br></div><div \
class="gmail_extra">FWIW, our QPA (or QPT, actually) already depends on \
QtDBus.</div><div class="gmail_extra"><br></div><div class="gmail_extra">

Aleix</div></div>



[prev in list] [next in list] [prev in thread] [next in thread]