-
Notifications
You must be signed in to change notification settings - Fork 690
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
Quick terminal splits get 1px out of sync with aggressive resizing #2944
Comments
Well that is extremely bizarre. It's not like we're setting the pixel-level height of the view lol, this is just a SwiftUI-contained view hierarchy. I didn't confirm it yet but I certainly see it in the video. Also very odd that hiding and re-showing fixes it, I'll have to think about all that. |
I just attempted to reproduce this in MacOS (14.6.1) and the default config and could not. It looks as expected, without the weird mismatch |
Thanks. I’ve done nothing to try to solve this yet. I haven’t yet been able to reproduce it. |
Yeah not a big deal. It's extremely low priority, and hopefully not even your problem but something in SwiftUI/AppKit or wherever. Mostly just confirming that I can still repro it with wildly-aggressive resizing (which is not at all like the resizing that I expect 99.99% of people to do). I don't even do such resizing normally–it was only when I was testing my TUI framework's parsing of in-band resizing notifications that caused me to discover it… |
Not sure if related but had this splitting at title-bar with zero config [video is normal speed]. Funnily enough, could not replicate when resizing from the left side. Ghostty 1.0.1 Screen.Recording.mov |
Here's another one.
Screen.Recording.2.mov |
I'm on macOS 14.7.1 (23H222) with Ghostty 9f75d93.
My full config is
I tend to keep the quick terminal split into two panes. Aggressive resizing causes them to get 1px out of sync in terms of height:
split.mov
Hiding and re-showing the quick terminal fixes this misalignment.
This issue occurs regardless of the
window-step-resize = true
settings.The text was updated successfully, but these errors were encountered: