You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
When inside the commit description part of the commit dialog, the default keybind to commit is Alt+Enter. This is unfortunate, because it's also bound to toggle fullscreen in Windows Terminal. I tried to change the keybind by setting keybind.universal.confirmInEditor to <c-enter>, but that is not a supported option.
Describe the solution you'd like
I'd prefer <c-enter> as the default keybind, this is a common UI pattern on the web. I'd also be fine to have it be configurable. I don't know whether the keybind.universal.confirmInEditor setting is used for this, but if so it should probably support <c-enter>.
#1700 might be related, but I believe that it's outdated as Alt+Enter is no longer used to enter newlines in the commit message, which you can now do using Enter itself.
I'd be happy to look into implementing this.
The text was updated successfully, but these errors were encountered:
This is very non-trivial. You'd first have to implement gdamore/tcell#671, and then add support for this in gocui.
Given the number of users having problems with this (see also #4375 (comment)) I'm wondering if we should change the default keybinding to something like <c-s> for the time being, to at least make it work everywhere. @jesseduffield Any opinion on that?
Is your feature request related to a problem? Please describe.
When inside the commit description part of the commit dialog, the default keybind to commit is
Alt+Enter
. This is unfortunate, because it's also bound to toggle fullscreen in Windows Terminal. I tried to change the keybind by settingkeybind.universal.confirmInEditor
to<c-enter>
, but that is not a supported option.Describe the solution you'd like
I'd prefer
<c-enter>
as the default keybind, this is a common UI pattern on the web. I'd also be fine to have it be configurable. I don't know whether thekeybind.universal.confirmInEditor
setting is used for this, but if so it should probably support<c-enter>
.#1700 might be related, but I believe that it's outdated as
Alt+Enter
is no longer used to enter newlines in the commit message, which you can now do usingEnter
itself.I'd be happy to look into implementing this.
The text was updated successfully, but these errors were encountered: