<table><tr><td style="">graesslin created this revision.<br />graesslin added reviewers: KWin, Plasma on Wayland.<br />Restricted Application added subscribers: kwin, plasma-devel.<br />Restricted Application added projects: Plasma on Wayland, 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/D2473" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>By default the InputRedirectionCursor is created and only the X11<br />
standalone platform creates the X11 specific cursor.</p>

<p>This is a preparation step for moving the X11 specific cursor<br />
implementation into the x11standalone platform plugin.</p>

<p>Move X11Cursor into the x11standalone platform plugin</p>

<p>At the same time the xinput2 integration is split out of X11Cursor<br />
and made a standalone part of the platform plugin. XInput integration<br />
is nowadays not only used by the cursor position polling, but also<br />
for modifier only shortcuts.</p>

<p>By splitting it out the modifier shortcuts start to work also when<br />
one doesn't have anything requesting a mouse position polling.</p>

<p>This also simplifies the conditional builds: xinput integration is<br />
only included if we have support for it at compile time without having<br />
to have many ifdefs in the cursor implementation. For the inclusion of<br />
cursor in the kcmkwin this also removes all the ifdefs.</p>

<p>The key events are only requested if we have xinput 2.1. Otherwise we<br />
would not get all raw events if the input device gets grabbed.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>x11cursor-to-platform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2473" rel="noreferrer">https://phabricator.kde.org/D2473</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
cursor.cpp<br />
cursor.h<br />
kcmkwin/kwinrules/CMakeLists.txt<br />
kcmkwin/kwinrules/detectwidget.cpp<br />
kcmkwin/kwinrules/kwinsrc.cpp<br />
keyboard_input.h<br />
main.cpp<br />
platform.cpp<br />
platform.h<br />
plugins/platforms/x11/standalone/CMakeLists.txt<br />
plugins/platforms/x11/standalone/x11_platform.cpp<br />
plugins/platforms/x11/standalone/x11_platform.h<br />
plugins/platforms/x11/standalone/x11cursor.cpp<br />
plugins/platforms/x11/standalone/x11cursor.h<br />
plugins/platforms/x11/standalone/xinputintegration.cpp<br />
plugins/platforms/x11/standalone/xinputintegration.h<br />
utils.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graesslin, KWin, Plasma on Wayland<br /><strong>Cc: </strong>plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas<br /></div>