-
-
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
PR: Fix Ctrl+Tab to cycle files on the editor #9400
Conversation
6ca6368
to
28f4e9a
Compare
I think they can go in the File menu. |
Like this @ccordoba12 ? |
Let's put them at the end, i.e. after |
👍 |
Ready @ccordoba12 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @goanpeca!
参考 spyder-ide/spyder#9400 核心思想是:Ctrl+Tab触发MenuBar上的Action,然后slot做具体的处理。Qt框架的eventFilter在mac上无法同时捕获Ctrl+Tab键。这个BUG很早就有了。但是一直无法很好的得到处理。 https://bugreports.qt.io/browse/QTBUG-8596 https://stackoverflow.com/questions/20694919/c-qt-osx-qtmetaqtkey-tab-shortcut-bind
Hi everyone. Did this eventually got fixed? I've downloaded and installed the latest version of Spyder 5.1.1. for Mac and can't see these newly added options to the file menu |
Description of Changes
As per https://bugreports.qt.io/browse/QTBUG-8596, the action needs to be added to a menu for this to work on OSX. Created a "hidden" menu to try this out and it is working :-). now... where should we add these actions? We cannot disable or hide them or this will cease to work
This PR now updates the default shortcut on MAC to use the
control
key as chrome firefox and similar applications do on OSX.There is a separate issue on the
Spyder Preferences
related to setting shortcuts on OSX that usetab
andcontrol
, which currently do not seem to work.Issue(s) Resolved
Fixes #6055
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct: @goanpeca