<table><tr><td style="">romangg accepted this revision.<br />romangg added a comment.<br />This revision is now accepted and ready to land.<br />Restricted Application edited projects, added Plasma; removed KWin.
</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/D10440" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Looks fine, but making this behavior dependent on <tt style="background: #ebebeb; font-size: 13px;">isDecorated</tt> seems a bit hacky to me. Wouldn't be a separate property for this better? Are all internal windows without decoration not movable? What if we want to do it this way at some point in the future? We just shouldn't forget about this check here then.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>modifier-click-scroll-decorated-internal-window-5.12</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10440" rel="noreferrer">https://phabricator.kde.org/D10440</a></div></div><br /><div><strong>To: </strong>graesslin, KWin, Plasma, romangg<br /><strong>Cc: </strong>romangg, plasma-devel, kwin, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, iodelay, bwowk, hardening<br /></div>