-
-
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: Restore submenu icon on internal console settings #13689
PR: Restore submenu icon on internal console settings #13689
Conversation
/show binder |
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.
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.
@goanpeca left a comment. Besides that LGTM 👍
514c16a
to
e1646c4
Compare
e1646c4
to
d6462e3
Compare
/show binder |
@steff456, there's no need to generate a new Binder button after new commits have been pushed to a PR. Any button will load its latest commits. |
@ccordoba12, thanks for the info! I didn't knew that any button will work (: |
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.
This PR is working as expected, it is ready to merge @dalthviz
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 LGTM 👍
Description of Changes
I think the Apple HIC guidelines do not recommend having Icons in SubMenus (they generally also advise against having submenus). So that is why this PR needs to override the default settings Qt uses for Mac with:
internal_settings_menu.menuAction().setIconVisibleInMenu(True)
Issue(s) Resolved
Fixes #13683
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