-
-
Notifications
You must be signed in to change notification settings - Fork 222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show the REPL/output with cursor at the bottom #792
Comments
I'd be fine with that the window always scrolls to bottom when opened via the command. Not sure what you mean by removing state, but if it works like you describe, I like it. |
This sounds good to me! |
This annoys me too, and |
Sounds odd that |
@PEZ When Calva Bindings are enabled, Ctrl-Home is bound to (default): Ctrl-End is bound to (default): |
Thanks! We should consider changing those bindings... |
Hi! 😄 This issue based on the fact that the REPL/output window can be easily closed, so we need a convenient way to open it again. There is a keyboard shortcut
ctrl+alt+c o
, which is very nice (and I use it a lot), but it has one drawback: the REPL/output opens with cursor at the top of the file. That means the user must manually scroll to the bottom to use the REPL.There is one special case, though, when
ctrl+alt+c o
brings back the REPL/output with cursor at the bottom: when the REPL/output contains unread evaluation results. It's implemented by introducing state:calva/src/results-output/results-doc.ts
Lines 245 to 253 in 5ff07b1
This auto-scrolling happens not only after using the keyboard shortcut, but also after opening the REPL/output file manually by clicking in the file explorer (not a big deal, but maybe a little bit magical).
I cannot imaging the case when it's necessary to open the REPL/output with cursor at the top, so my proposal is to eliminate the state and always bring it back with cursor at the bottom when using command/shortcut. The meaning of the command: "Show the REPL/output ready to be used". Manual clicking in the file explorer will still open the file in a regular and predictable way.
I've already made this change in my fork and it works very well for me. Will be happy to make a PR.
The text was updated successfully, but these errors were encountered: