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

Update keycap name for non-US backslash in Mac/Types.hs #766

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

postylem
Copy link
Contributor

@postylem postylem commented Nov 6, 2023

Change KeyNonUSBackslash to Key102nd to match the name defined in src/KMonad/Keyboard/Keycode.hs (they keycode which is assigned alias 'nubs').

Since Key102nd is assigned the the 'nubs' keycode (aka '102d', or 'lsgt') in Keycode.hs, it seems it should also be the keycode to which mac keycode (0x7, 0x64) (that is, the § key to the left of 1 on a UK-layout ISO keyboard) is assigned.

@postylem postylem changed the title Update keycap name for non-US backslash in Types.hs Update keycap name for non-US backslash in Mac/Types.hs Nov 6, 2023
In addition, change the old name, `KeyNonUSBackslash`, to `Key102nd`.
This matches the name defined in `src/KMonad/Keyboard/Keycode.hs` (the
keycode which is assigned the alias 'nubs').
@slotThe
Copy link
Member

slotThe commented Nov 17, 2023

Thanks!

@slotThe slotThe merged commit cec7d4a into kmonad:master Nov 17, 2023
13 checks passed
@postylem
Copy link
Contributor Author

postylem commented Nov 23, 2023

@slotThe I also had to modify the keymap/template/apple.kbd to work with my ISO macbook pro (shifting things over and adding this one additional key). Would it be helpful to submit a PR to add an keymap/template/apple-ISO.kbd ?

@postylem postylem deleted the patch-1 branch November 23, 2023 19:34
@tjwelde
Copy link

tjwelde commented Jan 25, 2024

I think that would be very helpful.

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.

None yet

3 participants