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

Ctrl + M to move a file to other panel isn't working #478

Closed
stevenxxiu opened this issue Jan 9, 2022 · 7 comments
Closed

Ctrl + M to move a file to other panel isn't working #478

stevenxxiu opened this issue Jan 9, 2022 · 7 comments
Labels
bug Something isn't working

Comments

@stevenxxiu
Copy link
Contributor

I'm trying to bind Ctrl + M to move a file to the other panel. This isn't working. Alt + M works though.

    {
        key: ctrl-m
        external: 'mv {file} {other-panel-directory}'
        leave_broot: false
    }

The escape sequence I have for this key is:

$ showkey -a

Press any keys - Ctrl-D will terminate this program

^[[109;5u        27 0033 0x1b
         91 0133 0x5b
         49 0061 0x31
         48 0060 0x30
         57 0071 0x39
         59 0073 0x3b
         53 0065 0x35
        117 0165 0x75
@stevenxxiu stevenxxiu added the bug Something isn't working label Jan 9, 2022
@Stargateur
Copy link
Contributor

are you sure ctrl+M is not catch by your terminal emulator ?

@Canop
Copy link
Owner

Canop commented Jan 9, 2022

As @Stargateur said, the problem isn't in broot: this is one of the many combinations that terminals intercept and can't be used in TUI.

@Canop
Copy link
Owner

Canop commented Jan 9, 2022

You can check that by running broot with the log and looking at what broot receives. See https://dystroy.org/broot/community/#log

@Canop Canop closed this as completed Jan 9, 2022
@stevenxxiu
Copy link
Contributor Author

stevenxxiu commented Jan 9, 2022

Indeed it appears that Broot sees nothing in the logs.

But Ctrl + M isn't caught by my terminal emulator (WezTerm), as showkey --ascii works when I press this key.

It might be caused by Support new terminal protocol for extended keys and more modifiers · Issue #569 · crossterm-rs/crossterm.

@Canop
Copy link
Owner

Canop commented Jan 9, 2022

Reopening for following the support of this new thing.

@Canop Canop reopened this Jan 9, 2022
@stevenxxiu
Copy link
Contributor Author

This works now, as crossterm-rs/crossterm#595 was merged and this is included in Crossterm v0.23.

@Canop
Copy link
Owner

Canop commented Jul 1, 2022

Thanks a lot for following this, @stevenxxiu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants