pindexis / marker

The terminal command palette
MIT License
2.02k stars 106 forks source link

Selection Bug #5

Open floscr opened 9 years ago

floscr commented 9 years ago

When trying to select something the screen doesn't clear

iTerm 2.1.1 OSX 10.10.2 Python 2.7.8 zsh 5.0.5 (x86_64-apple-darwin14.0)

pindexis commented 9 years ago

Hmm, it seems that the code responsible for clearing the screen is not working correctly in iTerm. I'll look into it as soon as I get hold on an OSX.

pindexis commented 9 years ago

Actually, this seems like an iterm bug. an ANSI escape code used for moving cursor vertically isn't working. I've filled an issue https://gitlab.com/gnachman/iterm2/issues/3617

simonweil commented 8 years ago

I have the same issue... Currently it is useless for me :( Well, I'll wait for iterm2 to update...

CaptainQuirk commented 8 years ago

I had the same issue right after I installed. I don't have the issue again today. I'm on iTerm 2.1.4

A colleague of mine does with the same version. Are you sure it is iTerm related ?

pindexis commented 8 years ago

I see that iterm 2.9 beta has been released which according to the author should fix the problem. https://www.iterm2.com/downloads.html

magpie514 commented 6 years ago

I'm getting a similar issue with konsole 17.04.3, in ubuntu 17.10 64b and bash 4.4.12 I solved it by changing the escape sequence for ERASE_SCREEN to ERASE_SCREEN = "\x1b[2J".

Since it's a different form of this bug it could be a good thing to keep a note of.

loirotte commented 5 years ago

I'm getting a similar issue with konsole 17.04.3, in ubuntu 17.10 64b and bash 4.4.12 I solved it by changing the escape sequence for ERASE_SCREEN to ERASE_SCREEN = "\x1b[2J".

Since it's a different form of this bug it could be a good thing to keep a note of.

Hello, same bug for me and I'm not sure to understand where you changed the definition for erasing the screen. Any clue? Thanks, Philippe

magpie514 commented 5 years ago

@loirotte Sorry! Didn't see your message until now. You have to go to line 5 in ansi.py (well it used to be line 5, I stopped using marker in favor of my own homebrewn tldr processor so it might be a bit out of date) and change ERASE_SCREEN to "\x1b[2J" (should be "\x1b[J" by default, just add the "2"). I hope that solves it for you.

loirotte commented 5 years ago

@hetdegon Thanks you your answer! It improves the display, but do not correct all issues. Sorry for my question, it finally seems obvious, but for some strange reason, I didn't look at the right place. I'm still searching for the whole solution, I will post if I find it!

intika commented 5 years ago

same issue here with konsole/qterminal/cool-retro-term... only work with xterm and real terminal... may be it's a qt bug...

ERASE_SCREEN = "\x1b[1J" is better... (but still not solved)

@pindexis i don't have the issue with qfc... any idea on this ?