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: Added missing close, focus and resize events #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rthrfrd
Copy link

@rthrfrd rthrfrd commented Aug 15, 2024

Thanks for the work to create this! Unfortunately it wouldn't fit my needs without some of these key events on Mac, so I've done my best to add them!

I added an NSWindowDelegate to the NSWindow to get more information about what's happening with the window (looking at the NSWindowDelegate API, there's more that can now be done here I'm sure). It emits NSEvents of type ApplicationDefined (as recommended by docs) back to the NSApplication to be picked up in your existing EventQueue.

I don't have much Objective-C experience so apologies in advance! I've tried to mirror the existing implementation/style as much as possible.

Added an NSWindowDelegate to the NSWindow to get more information. Emits ApplicationDefined NSEvents back to the NSApplication to be picked up in the existing EventQueue.
@alaingalvan
Copy link
Owner

Thanks Alex, this looks great, I'll go ahead and test it out on my end and merge it in after. 😁

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