Skip to content

[macOS] Hack to force window activation for non-bundled app.#17187

Merged
akien-mga merged 1 commit intogodotengine:masterfrom
bruvzg:macos_unbundled_app_focus_hack
Mar 3, 2018
Merged

[macOS] Hack to force window activation for non-bundled app.#17187
akien-mga merged 1 commit intogodotengine:masterfrom
bruvzg:macos_unbundled_app_focus_hack

Conversation

@bruvzg
Copy link
Copy Markdown
Member

@bruvzg bruvzg commented Mar 2, 2018

Fixes #8653.

@hpvb
Copy link
Copy Markdown
Member

hpvb commented Mar 3, 2018

0_o

Note to self for cherry-picking. Wait to see if anyone complains about this. I have no doubt that this hack works @bruvzg but I truly don't understand it :) and I have no way to test it. This probably should go into 3.0.z though.

@hpvb hpvb added the cherrypick label Mar 3, 2018
@akien-mga
Copy link
Copy Markdown
Member

If it works, that's fine by me. @bruvzg Could you add some comments in the code to explain why this hack is needed though?

@bruvzg bruvzg force-pushed the macos_unbundled_app_focus_hack branch from cc96bed to 506e17e Compare March 3, 2018 11:29
@bruvzg
Copy link
Copy Markdown
Member Author

bruvzg commented Mar 3, 2018

Added some comments.

AFAIK this is only way to fix unbundled apps focus on modern macOS (and unlike older hacks it's not using undocumented APIs), and it won't affect bundled apps at all.

@akien-mga akien-mga merged commit 26d42cd into godotengine:master Mar 3, 2018
@akien-mga
Copy link
Copy Markdown
Member

Thanks!

@toger5
Copy link
Copy Markdown
Contributor

toger5 commented Mar 3, 2018

Oh nice this is a great solution!!!
Thank you! so much for looking into it

@bruvzg bruvzg deleted the macos_unbundled_app_focus_hack branch March 3, 2018 14:57
@hpvb hpvb removed the cherrypick label Mar 28, 2018
@hpvb
Copy link
Copy Markdown
Member

hpvb commented Mar 28, 2018

Cherrypicked into 3.0.3

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