-
-
Notifications
You must be signed in to change notification settings - Fork 733
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
Keybind collision when using the colemak patch #1876
Comments
@thomashoullier do you (or any other colemak user) have any suggestion on what to change here? |
I use colemak-dh so h(as host) is ok for me. But that would clash with classic colemak. I suggest make it capital C. |
Thank you for the responses. I have no suggestion to make as I am not running colemak per-se and rebind many keys using my own patch. I simply noticed your colemak patch created the key collision in question and wanted to warn about the problem. |
@Darukutsu any suggestion for (EDIT: went with |
Environment details
NNN_TRASH=2
nnn
masterI compiled the latest nnn package revision from the gentoo repository with the
colemak
USE flag which enables the Colemak patch.I noticed the keybind for
SEL_NEW
in the patch (to the keyc
) clashed with the vanilla keybind forSEL_REMOTE
(also set toc
). This collision is detected bynnn -K
. When I ran nnn, thec
key was bound toSEL_REMOTE
and I could not access the "create" feature.I solved the problem on my installation by adding my own patch which removes the binding for
SEL_REMOTE
entirely. I don't think this solution is applicable to all colemak users. Maybe the colemak bindings patch needs to be adjusted so that bothSEL_REMOTE
andSEL_NEW
are accessible?Note the gentoo maintainers apply another patch to this patch, as seen here. I am not sure what this is about, but it seems related to the keybinds documentation only. This is probably a separate issue.
The text was updated successfully, but these errors were encountered: