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

Add tmux note to keybinding for list-keys (now safer) #454

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

eggbean
Copy link

@eggbean eggbean commented Sep 20, 2022

This PR simply adds a -N note so that the chosen key binding appears in tmux list-keys / <prefix> + ? and "Describe key binding" <prefix> + /, which is useful when you have a lot of custom keybindings.

@sturob
Copy link

sturob commented May 14, 2023

Using -N will fail to bind-keys on versions of tmux before 3.1 (changelog).

It is possible to add the note after the binding. This would fail in tmux < 3.2, but the binding will still work.

@eggbean
Copy link
Author

eggbean commented May 15, 2023

I knew it failed with tmux < v3.1, but as that came out three years ago and there are newer versions even in the RHEL and Debian repositories, I thought it would be reasonable to add it, but I didn't know about being able to add the note without it causing errors.

I don't have any easy way to test it, but is this better now?

@eggbean eggbean changed the title Add tmux note to keybinding for list-keys Add tmux note to keybinding for list-keys (now safer) May 15, 2023
dex6 added a commit to dex6/tmux-resurrect-more that referenced this pull request Jun 7, 2024
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