Notes from "New OSD in Plasma 2"

[Martin Gräßlin]

On Tuesday 21 January 2014 17:29:12 Martin Klapetek wrote:
> On Tue, Jan 21, 2014 at 3:28 PM, Martin Klapetek
> 
> <martin.klapetek at gmail.com>wrote:
> > On Tue, Jan 21, 2014 at 3:10 PM, Martin Gräßlin <mgraesslin at 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 <mgraesslin at 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140123/25f33ad7/attachment-0001.sig>