Add mappable action reverse_scroll_to_cursor - #4634
Conversation
|
I don't follow what's the use case for this action? |
|
For example, sometimes there will be some blank lines below the cursor at the end of autocomplete, so user can reverse scroll to move the cursor line back to the bottom of the screen. ( Yes, it would be better if these programs could support unscroll escape sequence. |
|
Maybe name it something better in that case, so it is more clear what it |
Ok, so far I just moved the line where the cursor is to the bottom, maybe I need to check all the empty lines and reverse scroll. changelog.rst
Is this Also I noticed that there are errors when generating documents. |
|
On Fri, Feb 04, 2022 at 03:07:29AM -0800, page-down wrote:
> Maybe name it something better in that case, so it is more clear what it does.
> Perhaps "scroll_prompt_to_bottom"
Ok, so far I just moved the line where the cursor is to the bottom, maybe I need to check all the empty lines and reverse scroll.
This doesn't seem to require a prompt check, does it?
No, it doesnt. Basically check for the last non-empty line after the
cursor and scroll it to the bottom.
Starting from the bottom, count all empty lines.
---
changelog.rst
``` :ac:`select_tab` ```
Is this `:ac:` something new? Or is it typo?
A typo.
Also I noticed that there are errors when generating documents.
```text
rm -rf docs/_build/
make dirhtml
shell-integration.rst:51: ERROR: Unknown interpreted text role "env".
shell-integration.rst:56: ERROR: Unknown interpreted text role "env".
```
These were fixed a while ago, are you up-to-date?
|
b162806 to
a1a0c9a
Compare
|
I need to check from the last line, since fish multi-line editing may have empty lines. Currently there is only one case when the last few lines of a fish multi-line edit are empty lines and will be cleared. I saw the latest
Indeed, I remember reading this |
For some programs that don't work well, they won't clear the blank lines under the cursor line. This action can be used to reverse the scrolling.
Also adjusted the name of the menu item that clears to the cursor
Cmd+K.Used initial capitalization and simplified the text (as it also works for the cursor prompt).
Moved Reset to the bottom, as this cleans up the most.
Please review if it is appropriate, thank you.