Skip to content
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

Feature request: scroll line up/down using keyboard shortcut #10968

Closed
jsh9 opened this issue Dec 7, 2019 · 8 comments · Fixed by #10990
Closed

Feature request: scroll line up/down using keyboard shortcut #10968

jsh9 opened this issue Dec 7, 2019 · 8 comments · Fixed by #10990

Comments

@jsh9
Copy link
Contributor

jsh9 commented Dec 7, 2019

Hi!

I've found that vscode and pycharm have a quite neat feature, which is to scroll down/up one line per key press, using ctrl + up/down.

Here is vscode's keyboard shortcut: https://code.visualstudio.com/docs/getstarted/keybindings (seacrh for "scrollLineDown"), and PyCharm has the same keyboard shortcut.

Currently, Spyder's ctrl + up/down is reserved for navigating between different code cells, so I wonder if we can have a line-by-line scroll assigned to another keyboard shortcut?

@ccordoba12
Copy link
Member

ccordoba12 commented Dec 8, 2019

Again, a nice suggestion of yours but unfortunately we don't have time to implement it. You're welcome to submit a PR for it though.

@jnsebgosselin
Copy link
Member

I think this would be a nice feature to have for the editor.

The hard part is to find a proper key sequence that everyone will agree with...

@jnsebgosselin
Copy link
Member

Maybe we could use Ctrl+Shift+Up and Ctrl+Shift+Down?

@goanpeca any thoughts?

@jsh9
Copy link
Contributor Author

jsh9 commented Dec 9, 2019

Psychologically, ctrl + up/down feels more incremental, and ctrl + shift + up/down feels more like a bigger step. If we are OK with breaking past user behaviors, I'd say assign ctrl + up/down to "scroll by line", and ctrl + shift + up/down to "navigate by cell".

@ccordoba12
Copy link
Member

@jsh9, you can easily remap shortcuts to your convenience, so I don't think that's necessary.

@jnsebgosselin
Copy link
Member

Ok I can take care of that one, I've tested it and it is simple to implement.

@ccordoba12
Copy link
Member

Sure, thanks a lot for your help @jnsebgosselin!

@ccordoba12 ccordoba12 modified the milestones: wishlist, v5.0beta1 Dec 9, 2019
@ccordoba12
Copy link
Member

I think new, non-critical features should go for Spyder 5. So please create your branch from master.

@goanpeca goanpeca removed this from the v5.0beta1 milestone Feb 18, 2020
@ccordoba12 ccordoba12 added this to the v5.0alpha2 milestone Aug 5, 2020
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha2, v5.0alpha3 Nov 12, 2020
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha3, v5.0alpha4 Jan 8, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha4, v5.0alpha5 Feb 14, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha5, v5.0alpha6 Feb 23, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha6, v5.0beta1 Mar 19, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0.0, v5.x Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants