D6591: XdgV6 - Kwin side

[David Edmundson]

davidedmundson commandeered this revision.
davidedmundson edited reviewers, added: mart; removed: davidedmundson.
davidedmundson added inline comments.
Restricted Application edited projects, added KWin; removed Plasma.

INLINE COMMENTS

> graesslin wrote in shell_client.cpp:277-280
> this needs to go through our PointerInputRedirection code, otherwise SeatInterface and PointerInputRedirection get out of sync. From my understanding that should be ::popupGrab?

Ack,

Initially I thought this event could happen at any time, but on re-reading it's to separate tooltips and menus, at which point it's on of the initial properties that should be sent before the first buffer. (I'll try to get that clarified in the spec for v7)

At which point it falls in nicely into how kwin already handles things checking hasPopupGrab when the toplevel is added.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D6591

To: davidedmundson, #plasma, graesslin, mart
Cc: mart, graesslin, kwin, plasma-devel, #kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170908/3e0dab0b/attachment.html>