On 20/Feb/2019 12:05, Junio C Hamano wrote:
> Ismael Luceno <ismael@xxxxxxxxxxx> writes:
>
> > - Control+BackSpace: Delete word to the left of the cursor.
> > - Control+Delete : Delete word to the right of the cursor.
> >
> > Originally introduced by BRIEF and Turbo Vision between 1985 and 1992,
> > they were adopted by most CUA-Compliant UIs, including those of: OS/2,
> > Windows, Mac OS, Qt, GTK, Open/Libre Office, Gecko, and GNU Emacs.
> >
> > Signed-off-by: Ismael Luceno <ismael@xxxxxxxxxxx>
> > ---
> >
> > Notes:
> > Changes since v1:
> > - Improved commit message.
>
> I do not use git-gui, but as a bystander, the intent of the change
> sounds sane. deleting from the insertion cursor to the end of the
> word where the insertion cursor is looks like the right way to
> delete the word to the right of the cursor. I am not sure if going
> back by one character from where the current insertion cursor is and
> further go back to the wordstart would give the beginning of the
> word to the left of the cursor, though.
Graphical explanation:
Deleting to the left:
v----- pointer
X_WORD_X
^--^------ selection
Deleting to the right:
v--------- pointer
X_WORD_X
^--^------ selection
Current implementation doesn't behave correctly when there's multiple
spaces:
Deleting to the left:
v--- pointer
X_WORD____X
^---- selection
Deleting to the right:
v--------- pointer
X____WORD_X
^--------- selection
> A larger issue is that we haven't heard from Pat, who has been
> helping git-gui.sh maintenance, for quite a while, so we'd need to
> find a volunteer to act as a replacement maintainer. If you are
> willing to, that would be great ;-)
Sure.
