Double clicking first output word selects rprompt #565

Closed
opened 4 months ago by RagnarGrootKoerkamp · 3 comments

When using fish shell, double clicking the first word in the output to select it also also selects the last word of the time/RPROMPT:

Expected behavior: only selection is selected
Actual behavior: both selection and 39 on the previous line are selected.

Note that this only happens with foot terminal, not with e.g. alacritty. On the other hand, ZSH in foot doesn't have this problem: when setting e.g. RPROMPT=abc and running echo selection, double clicking selection only selects that word and not the abc in the RPROMPT.

There should probably be some way to mark the end-of-line as a word separator.

I also reported this to fish, but according to them this should be solved by foot: https://github.com/fish-shell/fish-shell/issues/8037

When using `fish` shell, double clicking the first word in the output to select it also also selects the last word of the time/RPROMPT: <img src="https://ragnargrootkoerkamp.nl/upload/2021-05-31T11:47:03+02:00.xqs6lGrtQf.png"/> Expected behavior: only `selection` is selected Actual behavior: both `selection` and `39` on the previous line are selected. Note that this only happens with foot terminal, not with e.g. alacritty. On the other hand, ZSH in foot doesn't have this problem: when setting e.g. `RPROMPT=abc` and running `echo selection`, double clicking `selection` only selects that word and not the `abc` in the RPROMPT. There should probably be some way to mark the end-of-line as a word separator. I also reported this to fish, but according to them this should be solved by foot: https://github.com/fish-shell/fish-shell/issues/8037
dnkl added the
bug
label 4 months ago
Owner

Yeah, this is a bug in foot.

The word selection logic doesn't take explicit newlines into account, making it consider the rprompt a part of the first word on the next line.

Fix coming later today.

Yeah, this is a bug in foot. The word selection logic doesn't take explicit newlines into account, making it consider the rprompt a part of the first word on the next line. Fix coming later today.
dnkl closed this issue 4 months ago

So fast! :D Thanks!

So fast! :D Thanks!
Owner

Thanks for reporting!

Thanks for reporting!
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.