search

raphamorim / rio

A hardware-accelerated GPU terminal emulator focusing to run in desktops and browsers.
https://raphamorim.io/rio
MIT License
4.01k stars 129 forks source link

[Bug] sequence `CSI K` to work incorrectly #563

Open MrPandir opened 4 months ago

MrPandir commented 4 months ago

I tested the sequence \e[100m\e[K in rio it works differently than in other terminals. Tested in versions: 0.1.1 and 0.0.39.

Wiki CSI K

Videos

Rio

https://github.com/raphamorim/rio/assets/137798474/b277d7e1-041a-400e-a0c9-c8fe05904f76

Apple Terminal

https://github.com/raphamorim/rio/assets/137798474/831bc840-08ba-4929-9ed6-76cbb13fccd8

Kitty

https://github.com/raphamorim/rio/assets/137798474/ad17b76b-ab43-4dbf-b3eb-c61dbd09d531

raphamorim commented 4 months ago

Hey @MrPandir thanks for the issue,

I don't think this is connected to the CSI K but the way Rio renders lines, which is same as Alacritty (emulates same behaviour). Would you mind testing it in Alacritty to check?

MrPandir commented 4 months ago

I don't think this is connected to the CSI K but the way Rio renders lines, which is same as Alacritty (emulates same behaviour). Would you mind testing it in Alacritty to check?

Yes, it works on Alacritty just like in Rio. I think this is wrong behavior. And yes, this is a problem with the rendering method, I just don’t know what it’s called in terms of rendering in the terminal.

Alacritty

https://github.com/user-attachments/assets/4ff55067-02e5-4f0b-b8a1-ab8847231187