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

Linefeed Repo Maintaining #72

Open
kauri-hero opened this issue May 17, 2021 · 2 comments
Open

Linefeed Repo Maintaining #72

kauri-hero opened this issue May 17, 2021 · 2 comments

Comments

@kauri-hero
Copy link

kauri-hero commented May 17, 2021

Hello there @murarth
Our team has been using your linefeed library in our project and while its very good, it does have some bugs (seemingly centered around Windows machines) and would like to help resolve them, if you would be kind enough to provide some attention to this.

Our developers and users do enjoy the functionality that linefeed gives you, but these flaws certainly spoil the overall image of it. Also I’m convinced that our users would be of a similar mind.

I do see some of our issues are outlined on open Issue cards, so we will do our best to summarize below.

  1. Windows specific from what we can see - the terminal’s cursor stays statically printed on the screen at a certain line although we’ve already moved to another line (one or far away) with an active cursor, blinking.
    After you reset the terminal or scroll out of the area and come back then, you cannot see it anymore as the inner area of the terminal window has been re-rendered. I assume that it’s something to do with what you’re doing with hiding and showing the cursor (for example, at on our test, we saw some suspicious escape sequences in the output, which was supposed to be blank string; this pointed us to that thought too).

  2. Again mainly Windows, as far as we know - certain machines (the latest or modern Windows distributions) we experience unpleasantly slow backspacing. It’s most noticeable if you have long sequence of chars at the command line and you want to delete a bigger amount of them. If you just press backspace for a whole you will see that although you draw your finger back the cursor continue going back deleting some more characters. In this way you cannot control how many characters should be deleted exactly.

  3. Its preferred to give our users possibility to use some of those standard key combinations meant to tell to carry out certain action. For example, we’d like to have ctrl+c working. However, despite your library contains API for handling signals, these signals turn out unresponsive in the end. And this time it seems like it groups all platforms together (Linux, Windows, Mac) in this matter.
    Note: ctrl+d is the only signal that can be currently handled.

@murarth
Copy link
Owner

murarth commented May 17, 2021

I'm sorry, I no longer have the time to maintain this project or to make improvements. If you want to make improvements to linefeed, I welcome you to fork it and publish to crates.io under a new name.

@vi
Copy link

vi commented Nov 22, 2021

Maybe the fact that the library is unmaintained should be stated explicitly in README then?

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

3 participants