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

Fixes #1187: Cannot close the app from full screen mode on macOS #1291

Merged
merged 1 commit into from
Mar 22, 2024

Conversation

enesonus
Copy link


What's this PR do?
Fixes #1187. App was not closing at fullscreen mode at macOS. Changed the behavior for windows to leave fullscreen and then hide itself for macOS when pressed close button as many applications do so.

Any background context you want to provide?

Screenshots?
ezgif-4-f031ceac53

You have tested this PR on:

  • Windows
  • Linux/Ubuntu
  • macOS

@enesonus enesonus changed the title Fixes #1187 app not hiding at fullscreen in macOS Fixes #1187. Mar 31, 2023
@enesonus enesonus changed the title Fixes #1187. Fixes #1187 Mar 31, 2023
@cyrilchukwuebuka
Copy link

@enesonus
On mac os, the application is not meant to explicitly close when all the windows are closed
It's a native feature of Mac OS
Reference:

I hope the above helps out

@enesonus
Copy link
Author

enesonus commented Mar 31, 2023

@cyrilchukwuebuka this PR is not for quitting the app, it hides the window as you say so. Issue #1187 states that at fullscreen red x button does not hide or close the window i.e. does nothing. This PR checks if the window is fullscreen, makes the window not fullscreen and then hides it as the native behavior of most apps at macOS when red x is pressed.

@enesonus enesonus force-pushed the fix-1187-app-not-hiding branch 2 times, most recently from 3b27bcd to 2ed3de5 Compare April 2, 2023 14:54
@enesonus enesonus changed the title Fixes #1187 Fixes #1187: Cannot close the app from full screen mode on macOS Apr 2, 2023
@zulipbot zulipbot added size: L and removed size: S labels Jan 27, 2024
@zulipbot zulipbot added size: S and removed size: L labels Jan 27, 2024
@enesonus
Copy link
Author

Hi @andersk issue #1187 still seems to be persisting and this PR fixes it. Could you please give a review?

@andersk andersk merged commit 79f6f13 into zulip:main Mar 22, 2024
1 check passed
@andersk
Copy link
Member

andersk commented Mar 22, 2024

@enesonus I’ve edited your commit with a real commit message; this is important for maintaining a readable Git history.

Merged, thanks!

@enesonus
Copy link
Author

Thanks for the help @andersk! This was one of my first PRs and didn’t exactly know at that time how to write good commit messages.

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

Successfully merging this pull request may close these issues.

Cannot close the app from full screen mode on macOS
4 participants