Add Ctrl+C shortcut for clearing of current command

JustArchiNET / ASF-ui

The official web interface for ASF

Apache License 2.0

257 stars 36 forks source link

Add Ctrl+C shortcut for clearing of current command #1644

Closed TheMythologist closed 8 months ago

TheMythologist commented 9 months ago

Description

Adds a "Ctrl+C" shortcut to clear the current command in the terminal.

Checklist

[x] My pull request is not a duplicate
[x] I added a descriptive title to this pull request
[x] I added a concise description or a self-explanatory screenshot to this pull request
[x] My code follows the code style of this project
[x] I have performed a self-review of my own code

Aareksio commented 8 months ago

There were discussions about this in the past. I personally do not like introducing this shortcuts, as it risks introducing unexpected behavior for many casual users, gaining only small convenience to selected few power users. But it may very well not be the case.

MrBurrBurr commented 8 months ago

You could also use CTRL + A + CTRL + K to clear the current command, or not?

Otherwise I see it just like @Aareksio.

Maybe we should add another asf-ui config option 'Terminal PowerUser mode': true/false in order to enable all those shortcuts :p

Abrynos commented 8 months ago

Generally I like shortcuts. However, the thing with Ctrl+C is that (from how this shortcut usually works) one could expect that it cancels execution of a currently running command, which it definitely can NOT do. Therefore, I personally am not in favour of this.