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: Tabs are split into windows when using toggle_visibility action #4329

Open
mitchellh opened this issue Jan 1, 2025 · 1 comment
Open
Labels
gui GUI or app issue regardless of platform (i.e. Swift, GTK) os/macos
Milestone

Comments

@mitchellh
Copy link
Contributor

Discussed in #4240

Originally posted by writes-of-spring December 31, 2024
With 1.0.1 when running the toggle visibility command a window with tabs are made into multiple windows.

The toggle_visibility command is no longer configuration documentation in case it's no longer a featured option.

@mitchellh mitchellh added gui GUI or app issue regardless of platform (i.e. Swift, GTK) os/macos labels Jan 1, 2025
@dmehala
Copy link
Collaborator

dmehala commented Jan 4, 2025

I investigated a bit and the problem is here:

for window in NSApp.windows.filter({ $0.windowController is BaseTerminalController }) {
if isVisible {
window.orderOut(nil)
} else {
window.makeKeyAndOrderFront(nil)
}
}

orderOut(_:) has the side effect of removing windows from the tabGroup.

I am no swift expert so take this with a grain of salt. One possible solution could be to save the tabGroup state and restore it before bringing the window to the front

@mitchellh mitchellh added this to the 1.0.2 milestone Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gui GUI or app issue regardless of platform (i.e. Swift, GTK) os/macos
Projects
None yet
Development

No branches or pull requests

2 participants