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+X not working in nano #1069

Closed
WoodyWoodsta opened this Issue Dec 2, 2016 · 16 comments

Comments

Projects
None yet
@WoodyWoodsta

WoodyWoodsta commented Dec 2, 2016

Issue

As of be286c0, Ctrl+X does not work in nano, since it is being caught by hyper for the cut command. I'm sure this may be one of many conflicting keyboard shortcuts. Perhaps a way to temporarily bypass the hyper accelerators would help for these cases.

@natebwangsut

This comment has been minimized.

Show comment
Hide comment
@natebwangsut

natebwangsut Dec 2, 2016

For the reference, a8b17e6 still works with macOS using CTRL + X to quit the nano

natebwangsut commented Dec 2, 2016

For the reference, a8b17e6 still works with macOS using CTRL + X to quit the nano

@newgene newgene referenced this issue Dec 8, 2016

Closed

CTRL+a does not work in tmux under Linux #1100

2 of 2 tasks complete
@newgene

This comment has been minimized.

Show comment
Hide comment
@newgene

newgene Dec 8, 2016

I noticed that "CTRL+SHIFT+x" actually work for nano, in replace of "CTRL+x".

newgene commented Dec 8, 2016

I noticed that "CTRL+SHIFT+x" actually work for nano, in replace of "CTRL+x".

@radicaled

This comment has been minimized.

Show comment
Hide comment
@radicaled

radicaled Dec 12, 2016

Gnome Terminal has copy/paste keybindings set as "SHIFT+CTRL+X" and "SHIFT+CTRL+V" instead of CTRL+X/CTRL+V. On further inspection, every every commonly used short-cut in Gnome Terminal is prefixed with "SHIFT" to avoid conflicts with terminal applications that use the same keystrokes. Maybe that's the solution/convention Hyper should adopt for Linux?

radicaled commented Dec 12, 2016

Gnome Terminal has copy/paste keybindings set as "SHIFT+CTRL+X" and "SHIFT+CTRL+V" instead of CTRL+X/CTRL+V. On further inspection, every every commonly used short-cut in Gnome Terminal is prefixed with "SHIFT" to avoid conflicts with terminal applications that use the same keystrokes. Maybe that's the solution/convention Hyper should adopt for Linux?

@perry-mitchell

This comment has been minimized.

Show comment
Hide comment
@perry-mitchell

perry-mitchell Dec 13, 2016

For reference, this is still an issue on the 1.0.0 release..

perry-mitchell commented Dec 13, 2016

For reference, this is still an issue on the 1.0.0 release..

@Seldaek

This comment has been minimized.

Show comment
Hide comment
@Seldaek

Seldaek Dec 13, 2016

FWIW and because nobody mentioned it, the exact same problem exists on Windows as it also uses CTRL+* command bindings unlike OS X.

The best way forward I suppose would be to prefix Hyper-key bindings with CTRL+SHIFT rather than simple CTRL, or to add bindings configurability in the user config, so that users can remove all the unneeded shortcuts and/or rebind conflicting ones.

Seldaek commented Dec 13, 2016

FWIW and because nobody mentioned it, the exact same problem exists on Windows as it also uses CTRL+* command bindings unlike OS X.

The best way forward I suppose would be to prefix Hyper-key bindings with CTRL+SHIFT rather than simple CTRL, or to add bindings configurability in the user config, so that users can remove all the unneeded shortcuts and/or rebind conflicting ones.

@WoodyWoodsta

This comment has been minimized.

Show comment
Hide comment
@WoodyWoodsta

WoodyWoodsta Dec 13, 2016

I know that some of the key-bindings already use Shift and so it might be a little tricky. I was happy with the Ctrl+Super (or Win-key) as it was before. Perhaps this is grounds for a plugin that can switch between common sets of key-bindings found in other popular terminals.

WoodyWoodsta commented Dec 13, 2016

I know that some of the key-bindings already use Shift and so it might be a little tricky. I was happy with the Ctrl+Super (or Win-key) as it was before. Perhaps this is grounds for a plugin that can switch between common sets of key-bindings found in other popular terminals.

@perry-mitchell

This comment has been minimized.

Show comment
Hide comment
@perry-mitchell

perry-mitchell Dec 13, 2016

@WoodyWoodsta has a good point, though I would almost see the re-assignment of such highly-used shortcuts as a higher-priority when compared to support of customisable keybindings.

Ctrl+x is assigned to Cut, but the question is do more people use the Cut-shortcut than those who use nano and want to exit in the default (and displayed) way? What's more confounding is that Ctrl+k is assigned to "Clear" in Hyper, but this would act as Cut in nano... 😞

perry-mitchell commented Dec 13, 2016

@WoodyWoodsta has a good point, though I would almost see the re-assignment of such highly-used shortcuts as a higher-priority when compared to support of customisable keybindings.

Ctrl+x is assigned to Cut, but the question is do more people use the Cut-shortcut than those who use nano and want to exit in the default (and displayed) way? What's more confounding is that Ctrl+k is assigned to "Clear" in Hyper, but this would act as Cut in nano... 😞

@dfroze

This comment has been minimized.

Show comment
Hide comment
@dfroze

dfroze Dec 13, 2016

Hy, just started to play with this app. A quick question regarding this subjec, is there a way to change the shortcuts in hyper? It will be great if we could customize these shortcuts.

dfroze commented Dec 13, 2016

Hy, just started to play with this app. A quick question regarding this subjec, is there a way to change the shortcuts in hyper? It will be great if we could customize these shortcuts.

@nbolten

This comment has been minimized.

Show comment
Hide comment
@nbolten

nbolten Dec 16, 2016

Hi, I've put in a pull request that hopefully addresses this issue here: #1191 . It would make the default mod key into Ctrl+Shift on Windows and Linux so that there are no longer conflicts with CLI apps using Ctrl shortcuts.

It does not make the mod key configurable in hyper.js, but I think there's another patch in the works for that.

Feedback is helpful! The pull request is surely not perfect.

nbolten commented Dec 16, 2016

Hi, I've put in a pull request that hopefully addresses this issue here: #1191 . It would make the default mod key into Ctrl+Shift on Windows and Linux so that there are no longer conflicts with CLI apps using Ctrl shortcuts.

It does not make the mod key configurable in hyper.js, but I think there's another patch in the works for that.

Feedback is helpful! The pull request is surely not perfect.

@WoodyWoodsta WoodyWoodsta referenced this issue Dec 19, 2016

Closed

Can't exit to nano #1258

0 of 2 tasks complete

@nbolten nbolten referenced this issue Dec 20, 2016

Closed

Nano constant glitches #1271

2 of 2 tasks complete
@alexellis

This comment has been minimized.

Show comment
Hide comment
@alexellis

alexellis Apr 7, 2017

+1 this is an odd experience - for instance Control + W should cut back one word, but keeps closing my terminal. What's the best way to fix the bindings?

alexellis commented Apr 7, 2017

+1 this is an odd experience - for instance Control + W should cut back one word, but keeps closing my terminal. What's the best way to fix the bindings?

This was referenced Apr 30, 2017

@taylorthurlow

This comment has been minimized.

Show comment
Hide comment
@taylorthurlow

taylorthurlow May 6, 2017

Chiming in to agree that something should be done to preserve the ctrl+w shortcut.

taylorthurlow commented May 6, 2017

Chiming in to agree that something should be done to preserve the ctrl+w shortcut.

@ppot

This comment has been minimized.

Show comment
Hide comment
@ppot

ppot May 7, 2017

Collaborator

@taylorthurlow #1509 Already address this.

Collaborator

ppot commented May 7, 2017

@taylorthurlow #1509 Already address this.

@taylorthurlow

This comment has been minimized.

Show comment
Hide comment
@taylorthurlow

taylorthurlow May 7, 2017

@ppot Ah didn't think to check that. Glad to hear, thanks!

taylorthurlow commented May 7, 2017

@ppot Ah didn't think to check that. Glad to hear, thanks!

@ppot

This comment has been minimized.

Show comment
Hide comment
@ppot

ppot May 7, 2017

Collaborator

@taylorthurlow Nothing for!

Collaborator

ppot commented May 7, 2017

@taylorthurlow Nothing for!

@ppot

This comment has been minimized.

Show comment
Hide comment
@ppot

ppot Aug 15, 2017

Collaborator

close in #1876

Collaborator

ppot commented Aug 15, 2017

close in #1876

@ppot ppot closed this Aug 15, 2017

@jancborchardt

This comment has been minimized.

Show comment
Hide comment
@jancborchardt

jancborchardt Aug 15, 2017

Thanks a lot @ppot! :)

jancborchardt commented Aug 15, 2017

Thanks a lot @ppot! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment