Skip to content

GTK: Improve Split Close Behavior#11173

Open
cmwetherell wants to merge 4 commits into
ghostty-org:mainfrom
cmwetherell:close-split-gtk
Open

GTK: Improve Split Close Behavior#11173
cmwetherell wants to merge 4 commits into
ghostty-org:mainfrom
cmwetherell:close-split-gtk

Conversation

@cmwetherell
Copy link
Copy Markdown

@cmwetherell cmwetherell commented Mar 5, 2026

Summary

  • Adds a "Close Split" option to the right-click context menu in the split submenu
  • Allows users to close the focused split pane directly from the context menu

Reference discussion: #10982

cmwetherell and others added 2 commits March 4, 2026 16:27
Add a "Close Split" option to the split submenu in the right-click
context menu, allowing users to close the focused split pane directly
from the context menu.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Rename internal action and function from close-pane/actionClosePane
to close-split/actionCloseSplit for consistency with the UI label.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cmwetherell cmwetherell marked this pull request as ready for review March 5, 2026 00:38
@cmwetherell cmwetherell requested a review from a team as a code owner March 5, 2026 00:38
@jcollie
Copy link
Copy Markdown
Member

jcollie commented Mar 5, 2026

You'll need to regenerate the translations - run zig build update-translations and commit the results.

Copy link
Copy Markdown
Member

@pluiedev pluiedev left a comment

Choose a reason for hiding this comment

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

Diff LGTM sans jcollie's comment on regenerating translation strings. This will definitely not make it in 1.3, though, since new strings are introduced - will merge once 1.3 is released and branched off

@mitchellh mitchellh added this to the 1.4.0 milestone Mar 5, 2026
@cmwetherell
Copy link
Copy Markdown
Author

You'll need to regenerate the translations - run zig build update-translations and commit the results.

done!

@cmwetherell cmwetherell changed the title Improve Split Close Behavior | GTK GTK: Improve Split Close Behavior Mar 5, 2026
@tristan957
Copy link
Copy Markdown
Member

Looks like this needs a regeneration. Other than that, is this good to merge @jcollie @pluiedev?

Copy link
Copy Markdown
Member

@pluiedev pluiedev left a comment

Choose a reason for hiding this comment

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

LGTM pending merge conflicts

@ghostty-bot ghostty-bot Bot added the gtk Issues exclusive to GTK which are not OS-specific label Mar 25, 2026
@cmwetherell
Copy link
Copy Markdown
Author

Anything needed from me before merge? Should consider merging the macOS PR at the same time as well.

Copy link
Copy Markdown
Member

@jcollie jcollie left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gtk Issues exclusive to GTK which are not OS-specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants