Bug 100709 - Map KEY_FAVORITES input event code
Summary: Map KEY_FAVORITES input event code
Status: RESOLVED FIXED
Alias: None
Product: xkeyboard-config
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: xkb
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-18 14:43 UTC by Christian Kellner
Modified: 2018-05-14 22:16 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Map evdev keycode KEY_FAVORITES to XF86Favorites (1.57 KB, patch)
2017-04-18 14:48 UTC, Christian Kellner
Details | Splinter Review
Map evdev keycode KEY_FAVORITES to XF86Favorites (1.62 KB, patch)
2018-05-02 08:59 UTC, Benjamin Berg
Details | Splinter Review

Description Christian Kellner 2017-04-18 14:43:33 UTC
KEY_FAVORITES (0x16c) is currently not mapped.
Comment 1 Christian Kellner 2017-04-18 14:48:10 UTC
Created attachment 130901 [details] [review]
Map evdev keycode KEY_FAVORITES to XF86Favorites

NB: The value of KEY_FAVORITES is 0x16c (364) and thus beyond 255,
which is not supported by X11. It might also need a recent xkbcomp
that is patched so it does not throw fatal errors for those. See
https://lists.x.org/archives/xorg-devel/2017-April/053389.html

https://bugs.freedesktop.org/show_bug.cgi?id=100709

Signed-off-by: Christian Kellner <christian@kellner.me>
Comment 2 Sergey V. Udaltsov 2017-04-30 23:54:30 UTC
Is it safe from x11 POV to have codes >255 ?
Comment 3 Daniel Stone 2017-05-01 11:41:53 UTC
As long as xkbcomp (as in the link) is patched, then yes, it'll just be ignored. Without that fix, it's a fatal error.
Comment 4 Sergey V. Udaltsov 2017-05-01 11:59:32 UTC
Is the current version of xorg patched? Or is it only in DEV branch? I would not like to have that included before the stable xorg can handle it.
Comment 5 Daniel Stone 2017-05-01 14:37:47 UTC
There hasn't been a combined 'katamari' release since 7.7, which was 5 years ago now. I don't expect there will be another one either. But I guess we could contact all the distros individually and ask them to make sure they update xkbcomp. How long would you like to wait, or would it be based on xkbcomp entering particular distros, or ... ?
Comment 6 Sergey V. Udaltsov 2017-05-01 15:54:26 UTC
Ok, hypocrisy aside, where are RedHat and Debian now in relation to that code?
Comment 7 Daniel Stone 2017-05-01 16:10:28 UTC
(In reply to Sergey V. Udaltsov from comment #6)
> Ok, hypocrisy aside, where are RedHat and Debian now in relation to that
> code?

Neither have shipped it :)
Comment 8 Sergey V. Udaltsov 2017-05-01 16:12:24 UTC
Perhaps I should wait a bit then... September release may be (considering that May release is going to happen soon)
Comment 9 Damjan Georgievski 2017-05-14 13:42:44 UTC
KEY_KEYBOARD too, please? :)

$ sudo libinput-debug-events --device /dev/input/event4
-event4   DEVICE_ADDED     ThinkPad Extra Buttons            seat0 default group1  cap:k
 event4   KEYBOARD_KEY      +1.69s      KEY_FAVORITES (364) pressed
 event4   KEYBOARD_KEY      +1.69s      KEY_FAVORITES (364) released
 event4   KEYBOARD_KEY      +1.95s      KEY_KEYBOARD (374) pressed
 event4   KEYBOARD_KEY      +1.95s      KEY_KEYBOARD (374) released
Comment 10 Christian Kellner 2017-06-20 15:52:44 UTC
(In reply to Damjan Georgievski from comment #9)
> KEY_KEYBOARD too, please? :)
I filed bug #101521 for that.
Comment 11 Benjamin Berg 2018-05-02 08:59:20 UTC
Created attachment 139262 [details] [review]
Map evdev keycode KEY_FAVORITES to XF86Favorites

Patch rebased to master
Comment 12 Sergey V. Udaltsov 2018-05-14 22:16:52 UTC
Thanks committed!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.