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

Using command + comma on macOS as default shortcut for editor settings #38451

Merged

Conversation

eduardonunesp
Copy link
Contributor

@eduardonunesp eduardonunesp commented May 4, 2020

This PR addresses a common shortcut on mac for preferences of any application, using the Commad + , is a default behavior for mac users

#7884

@Calinou
Copy link
Member

Calinou commented May 4, 2020

Thanks for opening a pull request 🙂

Note that this change was done in #38044 already: https://github.com/godotengine/godot/pull/38044/files#diff-b6d74ccf775245adb707953658e4defdR6853-R6857

@eduardonunesp
Copy link
Contributor Author

Hey @Calinou that's awesome, looks like the PR https://github.com/godotengine/godot/pull/38044/files#diff-b6d74ccf775245adb707953658e4defdR6853-R6857 is focusing master branch, right? I was testing only on 3.2 versions also because of the master branch didn't work on my mac for some reason (which is not related to this PR)

@Calinou
Copy link
Member

Calinou commented May 4, 2020

@eduardonunesp I didn't notice that, sorry. I guess it makes sense to merge this in the 3.2 branch then.

@Calinou Calinou added this to the 3.2 milestone May 4, 2020
editor/editor_node.cpp Show resolved Hide resolved
@akien-mga akien-mga requested a review from bruvzg May 11, 2020 09:41
@akien-mga akien-mga changed the base branch from 3.2 to master May 11, 2020 11:04
@akien-mga akien-mga force-pushed the feature/command-comma-preferences branch from 329990d to 75f77f7 Compare May 11, 2020 11:05
@akien-mga
Copy link
Member

Squashed commits together and rebased on master branch.

@akien-mga akien-mga modified the milestones: 3.2, 4.0 May 11, 2020
@akien-mga akien-mga added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label May 11, 2020
@akien-mga akien-mga merged commit 1a43d3b into godotengine:master May 16, 2020
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 3.2.2.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants