command-pal icon indicating copy to clipboard operation
command-pal copied to clipboard

Make shift+backspace erase entire search input

Open rouilj opened this issue 2 years ago • 1 comments

Rather than backspacing a bunch of times. Also it works from any position in the input.

This patch is arguable since the search terms should be short and hitting 4 or 5 backspaces isn't horrible. Also on some systems (windows) ctrl+backspace erases the previous word.

This needs documentation, but I am not sure where it should go in the README. Documenting the use of the return key to:

accept the currently highlighted entry or close command-pal if there is no match

The use of up/down arrows to navigate through the list.

Also if the "close command-pal on (multiple) backspaces" patch is added, documenting that would be good as well.

NOTE: if this code is merged after the multiple backspaces code, the then clause for this code should reset the backup counter and clear the feedback box like the other cases. Also it needs to go before the backspace exit code in the if/then chain otherwise it will never be reached if exit with backspace is enabled.

rouilj avatar Feb 10 '23 05:02 rouilj

Hi Ben:

Sorry I've been MIA. Have you given any more though to incorporating this? I am running my hacked version and would like to get these changes upstream.

-- rouilj

rouilj avatar Jun 05 '23 03:06 rouilj