-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Restore Ctrl+Return to run cells #10787
Comments
Thanks for noticing this small regression. We'll fix it after we release Spyder 4. |
I can take care of that one if you wish. |
Please go ahead @jnsebgosselin (side note, I will upload the translations tonight, sorry for the delays!) |
I can think of two options to solve this:
Or maybe there is another option. Any thoughts? |
We will do this for Spyder 5, which is something we wanted to do for some time now.
We should probably do this for now to avoid the regression. Thoughts @ccordoba12 ? |
@jnsebgosselin we talked a bit with @ccordoba12. Do you think you could work on this by adding an extra column to the shortcuts section on the preferences pane? The rationale is that having more than 2 shortcuts/aliases is too much so we can keep the interface simple by adding an extra column. We would also need to make the entries on the configuration lists and handle this appropriately. Thoughts? |
Yes, I agree with this, from my experience, having 2 configurable shortcuts per action seems to be the norm. I will start working on this 👍 |
Great! thanks a lot :-) |
@ccordoba12 @goanpeca Would it be ok to store them using a list of strings? This would make it very convenient for assigning shortcuts to |
That's fine for me and what I was about to suggest. |
@jnsebgosselin that looks awesome ! Is it possible to trigger the shortcut editor with the keyboard also? (instead of mouse click?) Please open a PR :-) |
Yes, it is still possible to trigger the shortcut editor with the keyboard when the table has focus. However, It was previously possible to navigate the table and trigger the shortcut editor while the search line edit had focus. I'm not sure how to handle that case with this design though. I don't know if it's a big deal, since returning the focus to the table can easily be done with a simple Shift-Tab. I'll open PR and we can discuss this over there. |
(imple Shift-Tab. yeah thats ok!) |
It looks pretty neat, thanks @jnsebgosselin! |
Moved to Spyder 4 RC2 - awesome work guys! The dark theme is super slick!
I miss something from previous Spyders though -
run cell
used to work with both the main bigReturn
, and the smallEnter
at the corner of theNumPad
, i.e. both of them mapped to the sameReturn
key in previous Spyders. But now they're differentiated.It's more convenient to be able to use both shortcuts for the same command. Now I can set both
Ctrl+Return
, andCtrl+Enter
for the shortcut, but Spyder interprets this as a sequence.The text was updated successfully, but these errors were encountered: