Rendering broken with split panes after zoom unzoom

[snaggen]

What Operating System(s) are you seeing this problem on?

Linux Wayland

Which Wayland compositor or X11 Window manager(s) are you using?

Mutter Wayland

WezTerm version

20230212-103106-a5c2b1f3

Did you try the latest nightly build to see if the issue is better (or worse!) than your current version?

Yes, and I updated the version box above to show the version of the nightly that I tried

Describe the bug

I can now, with a nightly I just installed break rendering, so the text are rendered over the split pane width.

1. Split a terminal into 2 panes.
2. zoom the left pane to fill the whole window.
3. cat a file with long lines that fill the window width.
4. unzoom back to the split view. Now the rendering is broken, so the lines are still rendered with full width.

Any resize event will fix the rendering issue.

On a upgraded Fedora 37 under wayland on a x86_64 machine.

To Reproduce

No response

Configuration

no config

Expected Behavior

No response

Logs

No response

Anything else?

No response

[kaddkaka]

It's exactly commit cc15f97 that causes this bug to appear. (From #3068)

Is it somewhat easy to write tests for this? I haven't acquainted myself with the codebase yet.

[wez]

This should be resolved now in main.

It typically takes about an hour before commits are available as nightly builds for all platforms. Linux builds are the fastest to build and are often available within about 20 minutes. Windows and macOS builds take a bit longer.

Please take a few moments to try out the fix and let me know how that works out. You can find the nightly downloads for your system in the wezterm installation docs.

If you prefer to use packages provided by your distribution or package manager of choice and don't want to replace that with a nightly download, keep in mind that you can download portable packages (eg: a .dmg file on macOS, a .zip file on Windows and an .AppImage file on Linux) that can be run without permanently installing or replacing an existing package, and can then simply be deleted once you no longer need them.

If you are eager and can build from source then you may be able to try this out more quickly.

[kaddkaka]

Works for me 👍

[snaggen]

I can confirm it works for me too!

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.