bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Dan Jacobson]

Here none of the "C-e ;; move-end-of-line" should be necessary,
as the position of the previous cursor should be remembered along
with the previous command.
Or the default position of the cursor should be the end of line,
not beginning. Just like comint-previous-input.

 !	  ;; dired-do-shell-command
 l	  ;; self-insert-command
 s	  ;; self-insert-command
 <return> ;; exit-minibuffer
 !	  ;; dired-do-shell-command
 <up>	  ;; previous-line-or-history-element
 C-e	  ;; move-end-of-line
 SPC	  ;; self-insert-command
 -	  ;; self-insert-command
 a	  ;; self-insert-command
 <return> ;; exit-minibuffer
 !	  ;; dired-do-shell-command
 <up>	  ;; previous-line-or-history-element
 C-e	  ;; move-end-of-line
 l	  ;; self-insert-command
 <return> ;; exit-minibuffer

emacs 29.3.

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Eli Zaretskii]

tags 71728 notabug wontfix
close 71728
thanks

> From: Dan Jacobson <jidanni@jidanni.org>
> Date: Sun, 23 Jun 2024 07:16:49 +0800
> 
> Here none of the "C-e ;; move-end-of-line" should be necessary,
> as the position of the previous cursor should be remembered along
> with the previous command.
> Or the default position of the cursor should be the end of line,
> not beginning. Just like comint-previous-input.
> 
>  !	  ;; dired-do-shell-command
>  l	  ;; self-insert-command
>  s	  ;; self-insert-command
>  <return> ;; exit-minibuffer
>  !	  ;; dired-do-shell-command
>  <up>	  ;; previous-line-or-history-element
>  C-e	  ;; move-end-of-line
>  SPC	  ;; self-insert-command
>  -	  ;; self-insert-command
>  a	  ;; self-insert-command
>  <return> ;; exit-minibuffer
>  !	  ;; dired-do-shell-command
>  <up>	  ;; previous-line-or-history-element
>  C-e	  ;; move-end-of-line
>  l	  ;; self-insert-command
>  <return> ;; exit-minibuffer

There's a possible conflict between moving the cursor vertically and
showing the previous command here.  If you don't like the current
heuristics, use M-p instead of the <UP> arrow.

I'm closing this bug because Emacs behaves here exactly as intended,
and changing that will reopen a large can of worms we had when the
current behavior was implemented.

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Dan Jacobson]

>>>>> "EZ" == Eli Zaretskii <eliz@gnu.org> writes:
>> From: Dan Jacobson <jidanni@jidanni.org>
>> !	  ;; dired-do-shell-command
>> l	  ;; self-insert-command
>> s	  ;; self-insert-command
>> <return> ;; exit-minibuffer
>> !	  ;; dired-do-shell-command
>> <up>	  ;; previous-line-or-history-element
>> C-e	  ;; move-end-of-line
>> SPC	  ;; self-insert-command
>> -	  ;; self-insert-command
>> a	  ;; self-insert-command
>> <return> ;; exit-minibuffer
>> !	  ;; dired-do-shell-command
>> <up>	  ;; previous-line-or-history-element
>> C-e	  ;; move-end-of-line
>> l	  ;; self-insert-command
>> <return> ;; exit-minibuffer

EZ> If you don't like the current heuristics, use M-p instead of the <UP> arrow.

Alas, in my above example, M-p behaves exactly the same as <up>.

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Eli Zaretskii]

> From: Dan Jacobson <jidanni@jidanni.org>
> Cc: 71728@debbugs.gnu.org
> Date: Sun, 23 Jun 2024 21:38:23 +0800
> 
> >>>>> "EZ" == Eli Zaretskii <eliz@gnu.org> writes:
> >> From: Dan Jacobson <jidanni@jidanni.org>
> >> !	  ;; dired-do-shell-command
> >> l	  ;; self-insert-command
> >> s	  ;; self-insert-command
> >> <return> ;; exit-minibuffer
> >> !	  ;; dired-do-shell-command
> >> <up>	  ;; previous-line-or-history-element
> >> C-e	  ;; move-end-of-line
> >> SPC	  ;; self-insert-command
> >> -	  ;; self-insert-command
> >> a	  ;; self-insert-command
> >> <return> ;; exit-minibuffer
> >> !	  ;; dired-do-shell-command
> >> <up>	  ;; previous-line-or-history-element
> >> C-e	  ;; move-end-of-line
> >> l	  ;; self-insert-command
> >> <return> ;; exit-minibuffer
> 
> EZ> If you don't like the current heuristics, use M-p instead of the <UP> arrow.
> 
> Alas, in my above example, M-p behaves exactly the same as <up>.

So you are complaining about the usual Emacs behavior of vertical
motion, which keeps the column?  Why should Emacs go to EOL in this
particular case when it doesn't do that in any other buffer/window?

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Dan Jacobson]

>>>>> "EZ" == Eli Zaretskii <eliz@gnu.org> writes:
EZ> Why should Emacs go to EOL in this particular case when it doesn't
EZ> do that in any other buffer/window?

Works great here:
$ emacs -f shell
 l	    ;; self-insert-command
 s	    ;; self-insert-command
 <return>   ;; comint-send-input
 <escape> p ;; comint-previous-input

bug#71728: previous-line-or-history-element should put cursor at end, like comint-previous-input

[Eli Zaretskii]

> From: Dan Jacobson <jidanni@jidanni.org>
> Cc: 71728@debbugs.gnu.org
> Date: Sun, 23 Jun 2024 22:05:07 +0800
> 
> >>>>> "EZ" == Eli Zaretskii <eliz@gnu.org> writes:
> EZ> Why should Emacs go to EOL in this particular case when it doesn't
> EZ> do that in any other buffer/window?
> 
> Works great here:
> $ emacs -f shell
>  l	    ;; self-insert-command
>  s	    ;; self-insert-command
>  <return>   ;; comint-send-input
>  <escape> p ;; comint-previous-input

Not relevant to the issue at hand.  Minibuffer is not a comint buffer.