fix(time-picker): only emit change when full time when buttons hidden…

…, closing fix
tiaguinho · Aug 15, 2020 · 11d2f0e · 11d2f0e
1 parent 20513b6
commit 11d2f0e
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/lib/timer-picker/timer-picker.component.ts b/src/lib/timer-picker/timer-picker.component.ts
@@ -184,11 +184,10 @@ export class MccTimerPickerComponent {
       setTimeout(() => (this.focus = 'min'), 250);
     } else {
       this._minute = <MccTimerPickerMinute>value;
-    }
-
-    // if buttons are hidden, emit new event when value is changed
-    if (this._hideButtons) {
-      this.confirmSelectedTime();
+      // if buttons are hidden, emit new event when value is changed
+      if (this._hideButtons) {
+        this.confirmSelectedTime();
+      }
     }
   }
 
@@ -295,9 +294,6 @@ export class MccTimerPickerComponent {
 
     this.selected.emit(formated);
 
-    // only close automatically if button aren't hidden
-    if (!this._hideButtons) {
-      this._isOpen = false;
-    }
+    this._isOpen = false;
   }
 }