<table><tr><td style="">davidedmundson commandeered this revision.<br />davidedmundson edited reviewers, added: mart; removed: davidedmundson.<br />davidedmundson added inline comments.<br />Restricted Application edited projects, added KWin; removed Plasma.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6591" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D6591#inline-31723" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">graesslin</span> wrote in <span style="color: #4b4d51; font-weight: bold;">shell_client.cpp:277-280</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">this needs to go through our PointerInputRedirection code, otherwise SeatInterface and PointerInputRedirection get out of sync. From my understanding that should be ::popupGrab?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Ack,</p>

<p style="padding: 0; margin: 8px;">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)</p>

<p style="padding: 0; margin: 8px;">At which point it falls in nicely into how kwin already handles things checking hasPopupGrab when the toplevel is added.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6591" rel="noreferrer">https://phabricator.kde.org/D6591</a></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma, graesslin, mart<br /><strong>Cc: </strong>mart, graesslin, kwin, plasma-devel, KWin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol<br /></div>