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] Fullscreen fix #121

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Namaneo
Copy link
Contributor

@Namaneo Namaneo commented Aug 25, 2023

This PR has few goals:

  • It disables completely the tabs creation when a new window is opened from an already fullscreen-ed window. That behavior creates issues related to window creation and destroy, especially when creating window with a prepared MTY_Frame. IMO, it also felt weird to have very different visual results when calling the exact same entry function with the exact same parameters, depending on the state of the application.
  • It fixes a crash that was consistently reproduced when closing fullscreened windows. The [NSWindow close] function being asynchronous, some events can still be fired after the struct window has been free'd (here, a resize event). The internal context is now nullified and null-checks have been added in event handlers.

@Namaneo Namaneo marked this pull request as draft August 25, 2023 09:41
* Disable tabs creation when opening a window from a fullscreened window
* Fix crash when disconnecting while having a fullscreened child window
src/unix/apple/macosx/app.m Outdated Show resolved Hide resolved
@Namaneo Namaneo marked this pull request as ready for review September 18, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants