<table><tr><td style="">graesslin requested changes to this revision.<br />graesslin added a comment.<br />This revision now requires changes to proceed.
</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><div><p>Now it looks good to me!</p>

<p>What I would like to see is unit tests for all of that. This should be fairly simple as the tests are prepared to be run for different kind of shell surfaces. Many tests already have a _data method where it is set to be run for wl_shell and xdg_shell_unstable_v5. This should be easy to be extended for additional v6.</p></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>mart, Plasma, graesslin, davidedmundson<br /><strong>Cc: </strong>mart, graesslin, kwin, plasma-devel, KWin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>