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.

Sign up for GitHub

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

Jump to bottom

Change up/down for number to use initial value #367

Open

jackcannon wants to merge 1 commit into terkelg:master from jackcannon:master

jackcannon commented Aug 4, 2022

This is a small change to make the up/down behaviour a little bit more natural.

Use case:

const {xMovement} = await prompts(
  {
    type: 'number',
    name: 'xMovement',
    message: 'How should the asset move horizontally (x)?',
    initial: 0
  }
);

Repro steps

Run above example code
See initial 'grayed out' 0 where value is displayed
Without inputting any new values, press ↑ (up)

Expected result: input value to change in regards to the 0
Actual result: input value is set to -Infinity

New behaviour: pressing ↑ (up) sets the value (cyan) to 0, pressing ↑ (up) again changes the value to 1

Ditto for pressing ↓ (down)


          Change up/down for number to use initial value

06b79fc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment