D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. We don't need this now we have Albert's patch, right? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 To: xuetianweng,

D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

graesslin added a comment. In D3829#158947 , @ngraham wrote: > @graesslin, does this still need changes to be merge-worthy? Yes, we did not migrate back to xlib, so the port to xcb is still required. REPOSITORY R119 Plasma Desktop

D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

aacid added a comment. My proposal to fix the bug at https://phabricator.kde.org/D10498 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 To: xuetianweng, davidedmundson Cc: aacid, z3ntu, ngraham, knambiar, graesslin, plasma-devel, ZrenBot, progwolff,

D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

ngraham added a comment. @graesslin, does this still need changes to be merge-worthy? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 To: xuetianweng, davidedmundson Cc: z3ntu, ngraham, knambiar, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai,

D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

z3ntu added a comment. This patch works for me :) REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 To: xuetianweng, davidedmundson Cc: z3ntu, ngraham, knambiar, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

ngraham added a comment. What's next for this? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 To: xuetianweng, davidedmundson Cc: ngraham, knambiar, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas,

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

graesslin added a comment. In https://phabricator.kde.org/D3829#73066, @xuetianweng wrote: > In https://phabricator.kde.org/D3829#72924, @graesslin wrote: > > > I would say the overall problem is that this here still uses an XLib event queue. But Qt uses an xcb connection. I would

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

xuetianweng added a comment. In https://phabricator.kde.org/D3829#72924, @graesslin wrote: > I would say the overall problem is that this here still uses an XLib event queue. But Qt uses an xcb connection. I would say this needs a porting away from the XLib code and use Qt's XCB

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

graesslin added a comment. I would say the overall problem is that this here still uses an XLib event queue. But Qt uses an xcb connection. I would say this needs a porting away from the XLib code and use Qt's XCB connection and event handling. REPOSITORY R119 Plasma Desktop REVISION

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

xuetianweng added a comment. In https://phabricator.kde.org/D3829#71988, @graesslin wrote: > In https://phabricator.kde.org/D3829#71877, @xuetianweng wrote: > > > In https://phabricator.kde.org/D3829#71714, @graesslin wrote: > > > > > Could you explain the relationship to global

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

graesslin added a comment. In https://phabricator.kde.org/D3829#71877, @xuetianweng wrote: > In https://phabricator.kde.org/D3829#71714, @graesslin wrote: > > > Could you explain the relationship to global shortcuts? That sounds to me like a bug which needs to be fixed in

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

xuetianweng added a comment. In https://phabricator.kde.org/D3829#71724, @knambiar wrote: > Might this change fix this issue? https://bugs.kde.org/show_bug.cgi?id=370588 Yeah, I suspect so. I already put it in CCBUG, but I'd prefer reporter to confirm it. REPOSITORY R119

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

xuetianweng added a comment. In https://phabricator.kde.org/D3829#71714, @graesslin wrote: > Could you explain the relationship to global shortcuts? That sounds to me like a bug which needs to be fixed in kglobalaccel I don't really sure about this. So basically XlibNotifications

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

knambiar added a comment. Might this change fix this issue? https://bugs.kde.org/show_bug.cgi?id=370588 REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: xuetianweng,

[Differential] [Commented On] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

graesslin added a comment. Could you explain the relationship to global shortcuts? That sounds to me like a bug which needs to be fixed in kglobalaccel REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3829 EMAIL PREFERENCES

[Differential] [Request, 31 lines] D3829: [touchpad] Use a separate X11 Display to monitor the XInput event.

xuetianweng created this revision. xuetianweng added a reviewer: davidedmundson. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY For some reason, the XI event is not received if the change is from global shortcut. This