jgrulich updated this revision to Diff 49133.
jgrulich added a comment.
Drop keyboard support from this review
REPOSITORY
R127 KWayland
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D18114?vs=49087&id=49133
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D18114
AF
graesslin added a comment.
I'm the author of this interface and I consider it nowadays as a mistake to
have added it. If I would restart I would do the whole thing differently: input
events need to be injected through a kernel module in a dedicated input device.
It would need a user space co
graesslin added a comment.
I don't want to have key press/release in it. It has a reason that I didn't
add key press/release from the start. The problem is the mismatch of key events
and keysyms. Especially the kdeconnect use case for which this interface was
developed has this problem. Andr
jgrulich updated this revision to Diff 49087.
jgrulich added a comment.
Fix documentation
REPOSITORY
R127 KWayland
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D18114?vs=49082&id=49087
BRANCH
fake-input-keyboard-key
REVISION DETAIL
https://phabricator.kde.org/D18114
AFFECT
jgrulich added a comment.
I didn't intent to push this together, but when creating a branch from master
which already had first round of changes caused my second part of changes to be
pushed here. I hope you don't mind that. I'm doing this in a hurry hoping to
get exception from David Faure
jgrulich retitled this revision from "FakeInput: add support for pointer move
with absolute coordinates" to "FakeInput: add support for pointer move with
absolute coordinates and keyboard key press and release".
jgrulich edited the summary of this revision.
REPOSITORY
R127 KWayland
REVISION D