sebas accepted this revision. sebas added a reviewer: sebas. sebas added a comment. This revision is now accepted and ready to land.
Consider my comment, but up to you. Patch looks good otherwise. INLINE COMMENTS > wayland_backend.cpp:574 > + const QString title = i18nc("Title of nested KWin Wayland with Wayland > socket identifier as argument", > + "KDE Wayland Compositor (%1)", > waylandServer()->display()->socketName()); > + if (grab.isEmpty()) { Plasma perhaps, instead of KDE? REPOSITORY R108 KWin BRANCH wayland-platform-confined-pointer REVISION DETAIL https://phabricator.kde.org/D3488 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #kwin, #plasma_on_wayland, sebas Cc: sebas, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp