Skip to content

macOS: Handle toggle_maximize action #7030

Open
@pluiedev

Description

@pluiedev

Looks like we forgot to consider the toggle maximize action in the macOS apprt (the case for it is simply missing, even among the fallthrough cases:

case GHOSTTY_ACTION_CLOSE_ALL_WINDOWS:
fallthrough
case GHOSTTY_ACTION_TOGGLE_TAB_OVERVIEW:
fallthrough
case GHOSTTY_ACTION_TOGGLE_WINDOW_DECORATIONS:
fallthrough
case GHOSTTY_ACTION_PRESENT_TERMINAL:
fallthrough
case GHOSTTY_ACTION_SIZE_LIMIT:
fallthrough
case GHOSTTY_ACTION_QUIT_TIMER:
Ghostty.logger.info("known but unimplemented action action=\(action.tag.rawValue)")
return false
)... I wonder if we could implement some sort of validation to make sure we do handle all known enum cases.

I'm not sure if you could maximize a window on macOS à la Linux or Windows, but even then we should explicitly mark it as an unsupported but known action.

Metadata

Metadata

Assignees

No one assigned

    Labels

    contributor friendlyA well-scoped, approachable issue for someone looking to contributor.guiGUI or app issue regardless of platform (i.e. Swift, GTK)os/macos

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions