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

Clink prompt retains history items when using arrow keys to cycle through history #387

Closed
Avinm opened this issue Mar 24, 2016 · 7 comments

Comments

@Avinm
Copy link

Avinm commented Mar 24, 2016

For some history items clink retains the first 8 characters in the prompt when cycling the history with arrow keys. The retained characters wont go away with backspace and would seem to have become part of the PS1. Hitting enter to bring up the next line is the only way to get rid of these 8 sticky characters. Using Clink 0.4.7 on Win7 x64

@Avinm Avinm changed the title Clink Prompt retains history items when using arrow keys to cycle through history Clink prompt retains history items when using arrow keys to cycle through history Mar 24, 2016
@doraeric
Copy link

doraeric commented Apr 5, 2016

Maybe the same problem?
I guess it happens when there are some specific utf-8 texts. I have found this one "σ".
The WideCharToMultiByte function return its width as 2, but it is actually one column wide when displayed.
Here's the image. I pressed and to show histories in the first line.
I pasted 'σ' and pressed Backspace in the second line.
http://i.imgur.com/rhVS3Pm.gif

@electronicdreamer
Copy link

Not sure if it's linked to utf-8 characters.

My recent history has items like "git clone http://myproject" and "rm -rf myfolder" and as soon as I get to the second item in history, I have the bug.

@techeverri
Copy link

I have the same problem, but using Clink v0.4.6 on Win10 x64.

@Avinm
Copy link
Author

Avinm commented Apr 13, 2016

Seems to be related to having special chars in your $PROMPT, my prompt currently is
PROMPT=$E[m$E[32m$E]9;8;"USERNAME"$E@$E]9;8;"COMPUTERNAME"$E$S$E[92m$P$E[90m$_$E[90m$$$E[m$S$E]9;12$E
The issue seems to be absent with the default prompt supplied with clink

@Avinm
Copy link
Author

Avinm commented May 4, 2016

Could you release a build with this fix?

@w1mb0r
Copy link

w1mb0r commented May 30, 2016

Can you please release a build with this fix?

@w1mb0r
Copy link

w1mb0r commented Aug 17, 2016

release 0.4.8 was released on july 13. works great; issue is resolved.
thanks.

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

No branches or pull requests

5 participants