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

text cursor advance from sixel seems wrong #820

Closed
jerch opened this issue Sep 17, 2022 · 2 comments
Closed

text cursor advance from sixel seems wrong #820

jerch opened this issue Sep 17, 2022 · 2 comments
Labels
bug Something isn't working

Comments

@jerch
Copy link

jerch commented Sep 17, 2022

Contour Terminal version

0.3.3.204

Installer source

GitHub: release page

Operating System

Ubuntu 18 LTS

Architecture

x86-64

Other Software

No response

Steps to reproduce

  • run echo -e '\x1bPq$-$-$-$-\x1b\\' --> terminal advances full page (seems to be locked to yaml height value?)

Expected Behavior

The scriptlet inserts 4 empty sixel lines, thus 24px of vertical cursor advance. Terminal text cursor should advance by the same amount in cells. If vt340 cursor mode is enabled, the cursor should stop on the text line which covers the 24px from the original line.

Actual Behavior

On running the scriptlet terminal advances a full page (seems to be locked to yaml height value?).

Additional notes

NB: The test is furthermore degenerated in the sense that it does not place any pixel data. In contour this currently does not seem to make any difference (always shows full page advance). Note that most terminals are not sixel/vt340 conform for that no-pixel "image", as most dont move the cursor at all, which is wrong following the sixel spec - the text cursor should stop at the last active sixel line. (xterm seems to respect the empty image correctly, also need to fix it in xterm.js...)

Edit:
I stumbled over this behavior due to this script (can be used to test):
https://raw.githubusercontent.com/jerch/xterm-addon-image/1f68efe07d9404a73fafe5af4769faab42c9da1f/fixture/inspect_palette.sh

@jerch jerch added the bug Something isn't working label Sep 17, 2022
@Utkarsh-khambra
Copy link
Collaborator

Thanks for reporting this. I think this is similar to #809. Can you test the newest release to see if this bug still persist.

@jerch
Copy link
Author

jerch commented Sep 17, 2022

@Utkarsh-khambra Thx for the pointer - yes the main issue is resolved in latest release. 👍

Though echo -e '\x1bPq$-$-$-$-\x1b\\' is prolly still wrong (still waiting for confirmation on vt340 behavior, can do another issue by then).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants