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

[mac os x] ensures newly created window is drawn in front and active #20

Closed
wants to merge 1 commit into from

Conversation

butterflysky
Copy link

I noticed during my testing that when I run my code from a terminal window on Mac OS X and the terminal window has the focus, the newly drawn window ends up behind my terminal. This patch forces the newly drawn window to the foreground when it's drawn, regardless of which app has the focus.

@andlabs
Copy link
Owner

andlabs commented Apr 18, 2016

This will break .app bundles; see this. (If you want to continue this route, put the call in uiInit() instead. It only needs to be run once, after setting the activation policy.)

@butterflysky
Copy link
Author

Oh wow, thanks for showing me that, I learned something. I'll abandon this pull request for now and just get in the habit of building app bundles for my projects on os x. Sorry I didn't do more thorough research before covering the same ground you covered in the past!

@andlabs
Copy link
Owner

andlabs commented Apr 20, 2016

That's all right; don't feel bad :) The contributions are always welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants