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: add titled-visible-menu option to macos-non-native-fullscreen #5397

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

Conversation

sassanh
Copy link

@sassanh sassanh commented Jan 27, 2025

Non-native fullscreen has certain limitations at the moment regarding being truly fullscreen (taking all screen surface) and losing no functionality when activated. Currently, tab functionality is lost when non-native fullscreen is activated.

This commit introduces the titled-visible-menu mode for macOS non-native fullscreen. Like visible-menu mode, it hides the dock and uses its surface, leaving the menubar visible. This mode makes full use of the screen (except for the menubar) while retaining the tabbar’s functionality.

While a truly fullscreen non-native mode without feature loss is ideal, this implementation provides a functional alternative in the meantime.

@sassanh sassanh force-pushed the titled-visible-menu-fullscreen branch from d5c3dc5 to 91b4920 Compare January 27, 2025 15:53
@sassanh sassanh marked this pull request as ready for review January 27, 2025 15:53
@sassanh
Copy link
Author

sassanh commented Jan 28, 2025

This is how it looks:
image

@sassanh sassanh force-pushed the titled-visible-menu-fullscreen branch from 91b4920 to abd0fd9 Compare January 28, 2025 10:41
Copy link
Contributor

@mitchellh mitchellh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm okay with this compromise. One request that is mostly busy work...

include/ghostty.h Outdated Show resolved Hide resolved
macos/Sources/Helpers/Fullscreen.swift Outdated Show resolved Hide resolved
Non-native fullscreen has certain limitations at the moment regarding
being truly fullscreen (taking all screen surface) and losing no
functionality when activated. Currently, tab functionality is lost when
non-native fullscreen is activated.

This commit introduces the `titled-visible-menu` mode for macOS
non-native fullscreen. Like `visible-menu` mode, it hides the dock and
uses its surface, leaving the menubar visible. This mode makes full use
of the screen (except for the menubar) while retaining the tabbar’s
functionality.

While a truly fullscreen non-native mode without feature loss is ideal,
this implementation provides a functional alternative in the meantime.
@sassanh sassanh force-pushed the titled-visible-menu-fullscreen branch from abd0fd9 to 899e419 Compare January 30, 2025 02:37
@sassanh
Copy link
Author

sassanh commented Jan 30, 2025

I'm okay with this compromise. One request that is mostly busy work...

@mitchellh Thanks! I applied the changes.

@sassanh sassanh requested a review from mitchellh February 1, 2025 04:44
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