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 _______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel