Page MenuHomePhabricator
Differential D8871

Change only one keycode to avoid overwriting mappings of other keyboards.
Needs ReviewPublic
Actions

Authored by thierry1970 on May 9 2019, 7:02 AM.

Details

Reviewers
raster
Summary

Patch proposed by pascal@ordissimo.com
The concern appears when you have a physical keyboard and a virtual keyboard.
If you press the virtual keyboard and the ask key is not present in "Virtual core XTEST keyboard", the keyboard is unmapped and then remapped incorrectly.

  • switch to XTEST keyboard mapping (Send a FakeKeyEvent with an unsued keycode to cause a mapping change. This way, XStringToKeysym() and XKeysymToKeycode() will match the keyboard mapping that will be used in the final XFakeKeyEvents).
  • change only one keycode to avoid overwriting mappings of other keyboards
  • use the first unused keycode of the current keyboard mapping
  • revert the mapping change after few secondes (X clients need some time to catch the keysym)
  • revert all mapping changes in ecore_x_shutdown

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11385
Build 8637: arc lint + arc unit
thierry1970 created this revision.May 9 2019, 7:02 AM
Herald added a project: efl. · View Herald TranscriptMay 9 2019, 7:02 AM
Herald added subscribers: committers, reviewers, cedric. · View Herald Transcript

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

thierry1970 requested review of this revision.May 9 2019, 7:02 AM
thierry1970 retitled this revision from Change only one keycode to avoid overwriting mappings of other keyboards. to Change only one keycode to avoid overwriting mappings of other keyboards. .May 9 2019, 7:15 AM
thierry1970 edited the summary of this revision. (Show Details)
thierry1970 edited the summary of this revision. (Show Details)
thierry1970 added a subscriber: devilhorns.
thierry1970 updated this revision to Diff 22119.May 13 2019, 12:05 AM
thierry1970 retitled this revision from Change only one keycode to avoid overwriting mappings of other keyboards. to Change only one keycode to avoid overwriting mappings of other keyboards..
thierry1970 edited the summary of this revision. (Show Details)
thierry1970 updated this revision to Diff 22120.May 13 2019, 2:02 AM
thierry1970 edited the summary of this revision. (Show Details)
thierry1970 updated this revision to Diff 22125.May 13 2019, 7:18 AM
thierry1970 edited the summary of this revision. (Show Details)
devilhorns resigned from this revision.May 22 2019, 4:52 AM
devilhorns added a subscriber: bu5hm4n.

@bu5hm4n Why was I added as reviewer on this ? I don't deal with X stuff anymore, nor do I do key mapping stuffs lol

bu5hm4n added a comment.May 22 2019, 11:12 PM

Because you are probably the only one developing other things in this area, so it you look like a good candidate for a review :).

devilhorns edited reviewers, added: raster; removed: devilhorns.Wed, Jun 26, 11:12 AM

Revision Contents
Changeset List

Diff 22125

View Options

src/lib/ecore_x/ecore_x.c

Loading...
View Options

src/lib/ecore_x/ecore_x_private.h

Loading...
View Options

src/lib/ecore_x/ecore_x_test.c

Loading...