fix: retain number prompt default

SBoudrias · Jun 15, 2021 · 2bc4808 · 2bc4808
1 parent c4a3c23
commit 2bc4808
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/packages/inquirer/lib/prompts/base.js b/packages/inquirer/lib/prompts/base.js
@@ -150,8 +150,12 @@ class Prompt {
       this.opt.suffix +
       chalk.reset(' ');
 
-    // Append the default if available, and if question isn't answered
-    if (this.opt.default != null && this.status !== 'answered') {
+    // Append the default if available, and if question isn't touched/answered
+    if (
+      this.opt.default != null &&
+      this.status !== 'touched' &&
+      this.status !== 'answered'
+    ) {
       // If default password is supplied, hide it
       if (this.opt.type === 'password') {
         message += chalk.italic.dim('[hidden] ');

diff --git a/packages/inquirer/lib/prompts/input.js b/packages/inquirer/lib/prompts/input.js
@@ -101,10 +101,7 @@ class InputPrompt extends Base {
    */
 
   onKeypress() {
-    // If user press a key, just clear the default value
-    if (this.opt.default) {
-      this.opt.default = undefined;
-    }
+    this.state = 'touched';
 
     this.render();
   }

diff --git a/packages/inquirer/test/specs/prompts/number.js b/packages/inquirer/test/specs/prompts/number.js
@@ -111,6 +111,7 @@ describe('`number` prompt', () => {
       done();
     });
 
-    this.rl.emit('line', '');
+    this.rl.input.emit('keypress', 'a', { name: 'a' });
+    this.rl.emit('line');
   });
 });