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

Support keybindings with Enter #1480

Closed
nilscox opened this issue Jun 10, 2022 · 4 comments
Closed

Support keybindings with Enter #1480

nilscox opened this issue Jun 10, 2022 · 4 comments
Labels
duplicate This issue or pull request already exists

Comments

@nilscox
Copy link

nilscox commented Jun 10, 2022

Hello,

I am trying to add a keybinding to alt+enter, but I get an error when launching zellij:

Deserialization error: keybinds: data did not match any variant of untagged enum KeyActionUnbind at line 9 column 11

Here is the config I use:

keybinds:
    unbind: true
    normal:
        # ...
        - action: [ToggleFocusFullscreen,]
          key: [Alt: Enter]

As far as I understand, the "Enter" keyword isn't supported. Is there a way to bind a command to alt+enter?

zellij --version: zellij 0.30.0
alacritty --version: alacritty 0.10.1 ()

Let me know if you need any other information.
Thanks!

@a-kenji a-kenji added the duplicate This issue or pull request already exists label Jun 10, 2022
@a-kenji
Copy link
Contributor

a-kenji commented Jun 10, 2022

Thank you for reaching out! Have you tried using:

Alt: "\n"

?

Sadly the handling of keybindings is still somewhat limited for us,
here you can find an overview of supported keys: https://zellij.dev/documentation/keybindings-keys.html .

Edit: I just checked and we don't seem to support this configuration.

@nilscox
Copy link
Author

nilscox commented Jun 10, 2022

Indeed, using Alt: "\n" does not work, it's like the keybinding was not defined (but zellij starts without error).

I'll be using another keybinding for now, no worries. Do you plan to support this use case, or is it a limitation that may never have a solution?

@a-kenji
Copy link
Contributor

a-kenji commented Jun 10, 2022

Do you plan to support this use case

Yes, if it is feasible to do so. In this case, I think it should be.

or is it a limitation that may never have a solution?

I hope we get there in the near future, but that is not certain. It is definitely an issue that we wish to alleviate together with all the other keybindings that we do not support. For me personally, I want Ctrl: " " to be a thing.

I will close this for now, because there are quite a few keybinding issues open.
#351
#357

Please do reach out in the future again!

@a-kenji a-kenji closed this as completed Jun 10, 2022
@nilscox
Copy link
Author

nilscox commented Jun 10, 2022

Thanks for your reply and for making this great tool 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants