@vaadin/time-picker 23.1.0-rc2 → 23.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/time-picker",
-  "version": "23.1.0-rc2",
+  "version": "23.1.1",
   "publishConfig": {
     "access": "public"
   },
@@ -34,18 +34,18 @@
   ],
   "dependencies": {
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/combo-box": "23.1.0-rc2",
-    "@vaadin/component-base": "23.1.0-rc2",
-    "@vaadin/field-base": "23.1.0-rc2",
-    "@vaadin/input-container": "23.1.0-rc2",
-    "@vaadin/vaadin-lumo-styles": "23.1.0-rc2",
-    "@vaadin/vaadin-material-styles": "23.1.0-rc2",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-rc2"
+    "@vaadin/combo-box": "^23.1.1",
+    "@vaadin/component-base": "^23.1.1",
+    "@vaadin/field-base": "^23.1.1",
+    "@vaadin/input-container": "^23.1.1",
+    "@vaadin/vaadin-lumo-styles": "^23.1.1",
+    "@vaadin/vaadin-material-styles": "^23.1.1",
+    "@vaadin/vaadin-themable-mixin": "^23.1.1"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
     "@vaadin/testing-helpers": "^0.3.2",
     "sinon": "^13.0.2"
   },
-  "gitHead": "154c6782c42145fed21e443559fbe2d781ad3ec7"
+  "gitHead": "390458d6519433a2dd502cef90da48e84573a275"
 }

package/src/vaadin-time-picker-combo-box.js

@@ -14,6 +14,7 @@ import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mix
  * @extends HTMLElement
  * @mixes ComboBoxMixin
  * @mixes ThemableMixin
+ * @private
  */
 class TimePickerComboBox extends ComboBoxMixin(ThemableMixin(PolymerElement)) {
   static get is() {

package/src/vaadin-time-picker.js

@@ -106,6 +106,7 @@ class TimePicker extends PatternMixin(InputControlMixin(ThemableMixin(ElementMix
           value="{{_comboBoxValue}}"
           disabled="[[disabled]]"
           readonly="[[readonly]]"
+          clear-button-visible="[[clearButtonVisible]]"
           auto-open-disabled="[[autoOpenDisabled]]"
           position-target="[[_inputContainer]]"
           theme$="[[_theme]]"
@@ -304,7 +305,7 @@ class TimePicker extends PatternMixin(InputControlMixin(ThemableMixin(ElementMix
   }
 
   /**
-   * Used by `ClearButtonMixin` as a reference to the clear button element.
+   * Used by `InputControlMixin` as a reference to the clear button element.
    * @protected
    * @return {!HTMLElement}
    */
@@ -375,12 +376,13 @@ class TimePicker extends PatternMixin(InputControlMixin(ThemableMixin(ElementMix
   }
 
   /**
-   * Override an event listener from `ClearButtonMixin`
-   * to prevent clearing the input value on Esc key.
-   * @param {Event} event
+   * Override an event listener from `KeyboardMixin`.
+   * @param {!KeyboardEvent} e
    * @protected
    */
   _onKeyDown(e) {
+    super._onKeyDown(e);
+
     if (this.readonly || this.disabled || this.__dropdownItems.length) {
       return;
     }
@@ -394,6 +396,17 @@ class TimePicker extends PatternMixin(InputControlMixin(ThemableMixin(ElementMix
     }
   }
 
+  /**
+   * Override an event listener from `KeyboardMixin`.
+   * Do not call `super` in order to override clear
+   * button logic defined in `InputControlMixin`.
+   * @param {Event} event
+   * @protected
+   */
+  _onEscape() {
+    // Do nothing, the internal combo-box handles Escape.
+  }
+
   /** @private */
   __onArrowPressWithStep(step) {
     const objWithStep = this.__addStep(this.__getMsec(this.__memoValue), step, true);
@@ -646,7 +659,7 @@ class TimePicker extends PatternMixin(InputControlMixin(ThemableMixin(ElementMix
   }
 
   /**
-   * Override method inherited from `ClearButtonMixin`.
+   * Override method inherited from `InputControlMixin`.
    * @protected
    */
   _onClearButtonClick() {}