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

[macOS] Change the shortcut for Align Transform with View #94026

Merged

Conversation

passivestar
Copy link
Contributor

@passivestar passivestar commented Jul 7, 2024

Option-Command-M doesn't work on mac, it minimizes all windows of the active app. Changing to Option-Command-G, it's right next to align rotation with view

@passivestar passivestar requested a review from a team as a code owner July 7, 2024 11:09
@Chaosus Chaosus added this to the 4.3 milestone Jul 7, 2024
@KoBeWi KoBeWi modified the milestones: 4.3, 4.4 Jul 31, 2024
@Calinou
Copy link
Member

Calinou commented Oct 16, 2024

I suggest also supporting Ctrl + Alt + G on other platforms (while keeping Ctrl + Alt + M there), so it's less jarring if you switch between macOS and Windows/Linux frequently.

@passivestar passivestar force-pushed the mac-align-transform-shortcut branch from 3a5df4c to 891173b Compare October 16, 2024 17:52
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally on Linux, it works as expected.

@Calinou Calinou added the cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release label Nov 25, 2024
@Repiteo Repiteo merged commit 1dffdfd into godotengine:master Nov 26, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 26, 2024

Thanks!

@passivestar passivestar deleted the mac-align-transform-shortcut branch November 26, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release platform:macos topic:editor topic:porting topic:3d
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants