Re: system tray test?

2015-12-15 Thread René J . V . Bertin
René J. V. Bertin wrote:

> Sebastian Kügler wrote:
>> Seems to make sense to me. Could you file a review request for this?
> 
> Ok, will put that on my todo list :)

https://git.reviewboard.kde.org/r/126369/

:)

R.

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: system tray test?

2015-12-14 Thread René J . V . Bertin
Sebastian Kügler wrote:

>> 
>> I'd suggest replacing the line above with a sequence "m->addAction, m-
>> 
>> >addSeparator" on OS X, or do you have other suggestions?
> 
> Seems to make sense to me. Could you file a review request for this?

Ok, will put that on my todo list :)

Cheers,
R.

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: system tray test?

2015-12-11 Thread Sebastian Kügler
On Friday, December 11, 2015 12:19:51 PM René J.V. Bertin wrote:
> Does KF5 provide any classes above Qt's for putting up and controlling an
> icon (with/out menu) in the "system tray"? If so, is there a simple test
> app I can try?
> 
> Reason I'm asking: kwalletmanager5 isn't showing a systray interface like
> kwalletmanager(4) does. Not on OS X (where Qt5's systray does work), and on
> Linux under a KDE4 desktop I'm not sure what I'm getting.

Not sure what exactly you mean with "putting up and controlling an icon in the 
system tray", but we do have a status notifier test app, you can find it in: 
plasma-workspace/applets/systemtray/tests/statusnotifier

There are also some test programs in frameworks/knotifications/tests, you may 
want to look at.

Cheers,
-- 
sebas

http://www.kde.org | http://vizZzion.org

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: system tray test?

2015-12-11 Thread René J . V . Bertin
Sebastian Kügler wrote:

> Not sure what exactly you mean with "putting up and controlling an icon in the
> system tray", but we do have a status notifier test app, you can find it in:
> plasma-workspace/applets/systemtray/tests/statusnotifier

Heh, that must be because I asked the question in one of those "I should be 
doing something else right now, but I'd better handle this lest I forget it 
again" moments. I referred to KSystemTrayIcon, and I see that it's being 
replaced with KStatusNotifierItem.

> There are also some test programs in frameworks/knotifications/tests, you may
> want to look at.

Indeed. Thanks for pointing that out. kstatusnotifieritemtest mostly works. 
There are some complaints about missing icons though.

And I see there is the platform-specific issue with the tray menu's title item:

titleAction = m->addSection(qApp->windowIcon(), title);

will only add a separator on platforms that don't support texted separators 
(basically all except for X11 and maybe Wayland). Qt's position on this is 
"don't use addSection (unless you don't care about the text getting lost)" but 
I 
think this is an application where the text should definitely *not* get lost.

I'd suggest replacing the line above with a sequence "m->addAction, m-
>addSeparator" on OS X, or do you have other suggestions?

R



___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel