Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Emit HID event when keyboard brightness changed via fn+space #53

Draft
wants to merge 1 commit into
base: fwk-hx20-hx30-4410
Choose a base branch
from

Conversation

Jules-Bertholet
Copy link

@Jules-Bertholet Jules-Bertholet commented Nov 7, 2024

Currently, changing the keyboard backlight brightness via fn+space does not notify the OS in any way. Therefore, OS indicators like GNOME Shell's keyboard backlight slider will show the wrong values. This PR corrects this issue by emitting a "Keyboard Backlight Set Level" HID event on fn+space. (The backlight brightness is still changed directly in firmware, so functionality in e.g. firmware menu won’t be affected.)

@t-8ch
Copy link

t-8ch commented Nov 9, 2024

Could you explain how this event gets interpreted by Gnome?
The kernel doesn't seem to know to handle it.

@JohnAZoidberg JohnAZoidberg self-assigned this Nov 11, 2024
@Jules-Bertholet
Copy link
Author

Unfortunately, proper testing reveals that Linux does not currently handle this HID usage, so I am marking as draft.

@Jules-Bertholet Jules-Bertholet marked this pull request as draft November 11, 2024 15:07
Currently, changing the keyboard backlight brightness via fn+space
does not notify the OS in any way. Therefore, OS indicators
like GNOME Shell's keyboard backlight slider will show the wrong values.
This PR corrects this issue by emitting a "Keyboard Backlight Set Level"
HID event on fn+space.

Signed-off-by: Jules Bertholet <julesbertholet@quoi.xyz>
@JohnAZoidberg
Copy link
Member

This would be really awesome, I think I tried that a few months ago and neither Windows nor Linux supported it :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants