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

Add close_tab keybinding action for macOS #4395

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

Conversation

liby
Copy link
Contributor

@liby liby commented Jan 2, 2025

This PR adds support for a dedicated close_tab keybinding action, allowing users to bind specific keys for closing tabs. The implementation:

  • Adds close_tab as a new keybinding action
  • Preserves all existing confirmation dialogs for running processes
  • Works seamlessly with macOS native tab system

Testing

  • Tested with single tabs
  • Tested with multiple tabs
  • Tested with running processes (confirmation dialog)
  • Tested with splits within tabs
image
demo.mov

Partially resolved #4331

@mitchellh
Copy link
Contributor

Resolves #4331

Just noting this is not true, since we have to also implement it for GTK. But I think there's a PR for that too.

Thanks for doing this, I'll review soon!

@liby liby changed the title Add close_tab keybinding action Add close_tab keybinding action for macOS Jan 3, 2025
Implement `close_tab` keybinding action to close the current tab and all splits within that tab.
@liby liby force-pushed the feature/add-close-tab-keybind branch from a18c576 to 13e89fd Compare January 3, 2025 01:17
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.

Add close_tab keybinding action
2 participants