Skip to content

Fix half-qwerty layout and keyd's path in doc#1001

Open
Zwyx wants to merge 1 commit intorvaiya:masterfrom
Zwyx:fix-half-qwerty
Open

Fix half-qwerty layout and keyd's path in doc#1001
Zwyx wants to merge 1 commit intorvaiya:masterfrom
Zwyx:fix-half-qwerty

Conversation

@Zwyx
Copy link

@Zwyx Zwyx commented May 2, 2025

Hi!

Thank you so much for having made Keyd! I was using an XKB configuration which was not perfectly supported by every application. I recreated my layout using keyd, and it's now flawless.

I am so impressed by the level of quality you have put into keyd. Your way of managing hybrid modifiers is so clever (if I understand correctly, you send a fake control press to prevent a press of a modifier (for instance Alt) from having an effect (for instance focusing the menu bar of the current application)). Very clever.

Keyd offers many customisation possibilities, while having a configuration format easy to understand. Such a breath of fresh air after XKB!

Anyway, here's a small PR of various things I picked up along the way.


Fix Half-Qwerty layout Enter key

return was used instead of enter.

Fix keyd path in doc

From /usr/share/keyd to /usr/local/share/keyd, which is confirmed by PREFIX in the makefile.

Fix various minor typos

@Zwyx Zwyx changed the title Fix half-qwerty layout and keyd path in doc Fix half-qwerty layout and keyd's path in doc May 2, 2025
docs/keyd.scdoc Outdated
E.g.

ln -s /usr/share/keyd/keyd.compose ~/.XCompose
ln -s /usr/local/share/keyd/keyd.compose ~/.XCompose
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be /usr/share on most distros, /usr/local/ is conventionally used for packages installed from source. Other than that the PR looks good :).

@rvaiya
Copy link
Owner

rvaiya commented May 22, 2025

Thanks :). A lot of thought went into the design of modifier layers to make sure they didn't break existing modifier semantics (which most other tools don't seem to care about).

The PR looks good, but I would appreciate if you could squash it into a single commit and make the minor change mentioned above.

`return` was used instead of `enter`.

Also:
- fix action 1 & 2 inverted for `timeout`,
- fix various minor typos.
@Zwyx Zwyx force-pushed the fix-half-qwerty branch from 9a9d12a to 4b771f1 Compare May 24, 2025 01:16
@Zwyx
Copy link
Author

Zwyx commented May 24, 2025

No worries of course! I've reverted the addition of /local in the path (I'm surprised because the Makefile does contain it, but I'm definitely no expert), and I've squashed all the commits together 👍

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.

2 participants