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

Fix Synchronized Output (mode 2026) reporting #3884

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

Conversation

darrenburns
Copy link

@darrenburns darrenburns commented Dec 19, 2024

Hi, thank you for Zellij!

I help maintain Textual and some dependent TUI apps like Posting which make use of mode 2026.

I started using Zellij yesterday (loving it!) and noticed that the mode 2026 query response was not being parsed as expected and so the output could not be synchronized.

Looking into the code of Zellij, I think the response is missing a ? character. If I'm understanding correctly, response should be of the format CSI ? 2026 ; N $ y where N can be any value in the range 0-4 (inclusive).

References:

Before

Notice the tearing effect as the cursor moves through the list. It's particularly noticeable as the cursor moves over PATCH.

zellij-before-mode2026__85pct_smaller.mp4

After

On a build containing this fix I cannot reproduce the tearing effect:

zellij-mode2026-enabled__85pct_smaller.mp4

Possibly fixes #2794

The response to the 2026 mode query was missing a `?` character.

The response should be of the format `CSI ? 2026 ; N $ y` where N can
be any value in the range 0-4 inclusive.

References:

https://gist.github.com/christianparpart/d8a62cc1ab659194337d73e399004036
https://vt100.net/docs/vt510-rm/DECRPM.html
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.

Flickering in Nushell when used with Zellij and WezTerm
1 participant