On Tuesday 21 January 2014 17:29:12 Martin Klapetek wrote: > On Tue, Jan 21, 2014 at 3:28 PM, Martin Klapetek > > <martin.klape...@gmail.com>wrote: > > On Tue, Jan 21, 2014 at 3:10 PM, Martin Gräßlin <mgraess...@kde.org>wrote: > >> On Tuesday 21 January 2014 14:26:59 Martin Klapetek wrote: > >> > On Tue, Jan 21, 2014 at 8:57 AM, Martin Gräßlin <mgraess...@kde.org> > >> > >> wrote: > >> > > I had a look and the Plasma side seems fine. You can verify using > >> > >> xprop. > >> > >> > > My > >> > > > >> > > demo dialog got the notification type. > >> > > >> > Hmm...I might have been wrong then, using xprop on the OSD I get > >> > > >> > _NET_WM_WINDOW_TYPE(ATOM) = _KDE_NET_WM_WINDOW_TYPE_OVERRIDE, > >> > _NET_WM_WINDOW_TYPE_NORMAL > >> > >> That's not what I had. My example code which I wrote this morning for > >> testing: > >> http://pastebin.kde.org/phhnfbtda > >> > >> I executed the code using qmlscene. Maybe that makes a difference. > > > > Ok, two things I noticed: > > > > 1) If "visible: true" is not present, the type is never NET::Notification, > > even in your code. Once added, it has proper type. > > 2) This works when executed in qmlscene only, it does not work when > > using KDeclarative::QmlObject for the dialog. > > > > I'll investigate why. > > Update: windowType for the Dialog is overiden by Qt > (in QXcbWindow::setNetWmWindowFlags). That method (in combination with > window flags) however does not support NET::Notification. Should we extend > that system? Or should we make that method not /replace/ the type but just > add it (would that even work?)? Should we work around that in our code?
Just for the record: it is a bug in PlasmaCore.Dialog - see https://git.reviewboard.kde.org/r/115258/ Cheers Martin G.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
