system tray test?

[Sebastian Kügler]

On Friday, December 11, 2015 08:09:25 PM René J. V. Bertin wrote:
> 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?

Seems to make sense to me. Could you file a review request for this?
-- 
sebas

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