<table><tr><td style="">graesslin created this revision.<br />graesslin added reviewers: KWin, Plasma.<br />Restricted Application added a project: KWin.<br />Restricted Application added subscribers: kwin, plasma-devel.</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/D2506" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The raw pointer button events intercepted in the XInput2 input filter<br />
get sent through the Platform to the PointerInputRedirection. This<br />
makes the PointerInputRedirection track the pointer button state and<br />
emit the signals for button changed and axis changed.</p>

<p>These signals are used by the modifier-only shortcut detection to<br />
determine whether the shortcut should trigger.</p>

<p>On X11 the "normal" input handling doesn't use the InputRedirection<br />
and the emitted signals are not consumed by anything else. As<br />
PointerInputRedirection is not inited the events are not forwarded<br />
to the input filter, thus won't be processed by other parts and<br />
won't interfere with the normal event processing on X11.</p>

<p>Given that it also doesn't matter that the input filter does not<br />
apply the left-handed setting. The internal tracking will have a<br />
wrong mouse button, but nothing is going to do decisions based on<br />
the value of the pressed mouse button. For the moment all we are<br />
interested in is that a button is pressed.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Pressed meta, clicked, scrolled, released meta: launcher<br />
did not open. Pressed meta, released meta: launcher opened</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>x11-buttons-to-platform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2506" rel="noreferrer">https://phabricator.kde.org/D2506</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/platforms/x11/standalone/xinputintegration.cpp<br />
pointer_input.cpp</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<br /><strong>Cc: </strong>plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas<br /></div>