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

Allow for backspacing when entering a password #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ning-github
Copy link

Before: When typing into a prompt.password, if you were to make a typo and press delete to on your keyboard so that you could change it, it would just keep adding to the buffer as well as the process.stdout.

After: The backspace keypress is caught, trimming the final character off the keypress, as well as not including it in the buffer.

Before: When typing into a prompt.password, if you were to make a typo and press delete to on your keyboard so that you could change it, it would just keep adding to the buffer as well as the process.stdout.

After: The backspace keypress is caught, trimming the final character off the keypress, as well as not including it in the buffer.
@jesusgp22
Copy link

works perfectly, but it doesn't remove the mask (*) character on the console, is this posible?

@zachfeldman
Copy link

Anyone planning to merge this soon? We're depending on it for a fix over at lambtron/emojipacks#72 :D

@iamleet
Copy link

iamleet commented Oct 4, 2017

This bug tho -_- , thanks for the pr dude.

@Ochaun
Copy link

Ochaun commented Jan 18, 2019

This really needs to be merged

@iamleet
Copy link

iamleet commented Jan 18, 2019

@OEMarshall dude, I would really give up on this and look into https://www.npmjs.com/package/prompts

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

Successfully merging this pull request may close these issues.

5 participants