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

Prepend 'XF86' where needed for audio/backlight #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Prepend 'XF86' where needed for audio/backlight #14

wants to merge 1 commit into from

Conversation

natemaia
Copy link

@natemaia natemaia commented Nov 10, 2018

When clicking in the 'Key' region for a binding it prompts you to enter a new accelerator, entering any dedicated key for audio or backlight will give you AudioRaiseVolume, MonBrightnessUp, so on and so forth. If you have dedicated volume or backlight keys then the bound keys need to be XF86AudioRaiseVolume, XF86AudioLowerVolume, XF86MonBrightnessUp, XF86MonBrightnessDown, etc..

Please correct me if I am wrong.

The key is returned by key_gtk2openbox(), this patch simply prepends XF86 when the key name has 'Audio' or 'MonBrightness' in it.

I'm sure there could be a more fancy/robust way of doing this (dict, etc), however this works and I don't believe it will have any collisions with other mappings/keys.

Cheers

When clicking in the 'Key' region for a binding it prompts you to enter a new accelerator, entering any dedicated key for audio or backlight will give you `AudioRaiseVolume`, `MonBrightness`, so on and so forth. If you have dedicated volume or backlight keys then the bound keys need to be `XF86AudioRaiseVolume`, `XF86AudioLowerVolume`, `XF86MonBrightnessUp`, `XF86MonBrightnessDown`, etc.. 

Please correct me if I am wrong.


The key is returned by `key_gtk2openbox()`, this patch simply prepends `XF86` when the key name has 'Audio' or 'MonBrightness' in it.
I'm sure there could be a more fancy way of doing this however this works and I don't believe it will have any collisions with other mappings/keys.

Cheers
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.

1 participant