romangg accepted this revision. romangg added a comment. This revision is now accepted and ready to land. Restricted Application edited projects, added Plasma; removed KWin.
Looks fine, but making this behavior dependent on `isDecorated` 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. REPOSITORY R108 KWin BRANCH modifier-click-scroll-decorated-internal-window-5.12 REVISION DETAIL https://phabricator.kde.org/D10440 To: graesslin, #kwin, #plasma, romangg Cc: romangg, plasma-devel, kwin, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, iodelay, bwowk, hardening
