@vaadin/date-picker 23.1.0-rc2 → 23.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/date-picker",
-  "version": "23.1.0-rc2",
+  "version": "23.1.1",
   "publishConfig": {
     "access": "public"
   },
@@ -34,19 +34,19 @@
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.3.0",
     "@polymer/polymer": "^3.2.0",
-    "@vaadin/button": "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-overlay": "23.1.0-rc2",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-rc2"
+    "@vaadin/button": "^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-overlay": "^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-date-picker-mixin.js

@@ -336,7 +336,7 @@ export const DatePickerMixin = (subclass) =>
     }
 
     /**
-     * Override a getter from `ClearButtonMixin` to make it optional
+     * Override a getter from `InputControlMixin` to make it optional
      * and to prevent warning when a clear button is missing,
      * for example when using <vaadin-date-picker-light>.
      * @protected
@@ -370,6 +370,7 @@ export const DatePickerMixin = (subclass) =>
     constructor() {
       super();
 
+      this._boundOnClick = this._onClick.bind(this);
       this._boundOnScroll = this._onScroll.bind(this);
     }
 
@@ -414,11 +415,7 @@ export const DatePickerMixin = (subclass) =>
     ready() {
       super.ready();
 
-      this.addEventListener('click', (e) => {
-        if (!this._isClearButton(e) && (!this.autoOpenDisabled || this._noInput)) {
-          this.open();
-        }
-      });
+      this.addEventListener('click', this._boundOnClick);
 
       this.addController(
         new MediaQueryController(this._fullscreenMediaQuery, (matches) => {
@@ -911,11 +908,35 @@ export const DatePickerMixin = (subclass) =>
     }
 
     /**
-     * Override an event listener from `ClearButtonMixin`
+     * @param {Event} event
+     * @private
+     */
+    _onClick(event) {
+      // Clear button click is handled in separate listener
+      // but bubbles to the host, so we need to ignore it.
+      if (!this._isClearButton(event)) {
+        this._onHostClick(event);
+      }
+    }
+
+    /**
+     * @param {Event} event
+     * @private
+     */
+    _onHostClick(event) {
+      if (!this.autoOpenDisabled || this._noInput) {
+        event.preventDefault();
+        this.open();
+      }
+    }
+
+    /**
+     * Override an event listener from `InputControlMixin`
      * to validate and dispatch change on clear.
      * @protected
      */
-    _onClearButtonClick() {
+    _onClearButtonClick(event) {
+      event.preventDefault();
       this.value = '';
       this._inputValue = '';
       this.validate();
@@ -924,11 +945,13 @@ export const DatePickerMixin = (subclass) =>
 
     /**
      * Override an event listener from `KeyboardMixin`.
-     * Do not call `super` to also override `ClearButtonMixin`.
+     * @param {KeyboardEvent} e
      * @protected
      * @override
      */
     _onKeyDown(e) {
+      super._onKeyDown(e);
+
       if (this._noInput) {
         // The input element cannot be readonly as it would conflict with
         // the required attribute. Both are not allowed on an input element.
@@ -955,28 +978,6 @@ export const DatePickerMixin = (subclass) =>
             this.open();
           }
           break;
-        case 'Enter': {
-          const parsedDate = this._getParsedDate();
-          const isValidDate = this._isValidDate(parsedDate);
-          if (this.opened) {
-            if (this._overlayInitialized && this._overlayContent.focusedDate && isValidDate) {
-              this._selectDate(this._overlayContent.focusedDate);
-            }
-            this.close();
-          } else if (!isValidDate && this.inputElement.value !== '') {
-            this.validate();
-          } else {
-            const oldValue = this.value;
-            this._selectParsedOrFocusedDate();
-            if (oldValue === this.value) {
-              this.validate();
-            }
-          }
-          break;
-        }
-        case 'Escape':
-          this._onEscape();
-          break;
         case 'Tab':
           if (this.opened) {
             e.preventDefault();
@@ -995,16 +996,56 @@ export const DatePickerMixin = (subclass) =>
       }
     }
 
-    /** @private */
-    _onEscape() {
+    /**
+     * Override an event listener from `KeyboardMixin`.
+     *
+     * @param {!KeyboardEvent} _event
+     * @protected
+     * @override
+     */
+    _onEnter(_event) {
+      const parsedDate = this._getParsedDate();
+      const isValidDate = this._isValidDate(parsedDate);
+      if (this.opened) {
+        if (this._overlayInitialized && this._overlayContent.focusedDate && isValidDate) {
+          this._selectDate(this._overlayContent.focusedDate);
+        }
+        this.close();
+      } else if (!isValidDate && this.inputElement.value !== '') {
+        this.validate();
+      } else {
+        const oldValue = this.value;
+        this._selectParsedOrFocusedDate();
+        if (oldValue === this.value) {
+          this.validate();
+        }
+      }
+    }
+
+    /**
+     * Override an event listener from `KeyboardMixin`.
+     * Do not call `super` in order to override clear
+     * button logic defined in `InputControlMixin`.
+     *
+     * @param {!KeyboardEvent} event
+     * @protected
+     * @override
+     */
+    _onEscape(event) {
       // Closing overlay is handled in vaadin-overlay-escape-press event listener.
       if (this.opened) {
         return;
       }
 
-      if (this.clearButtonVisible) {
-        this._onClearButtonClick();
-      } else if (this.autoOpenDisabled) {
+      if (this.clearButtonVisible && !!this.value) {
+        // Stop event from propagating to the host element
+        // to avoid closing dialog when clearing on Esc
+        event.stopPropagation();
+        this._onClearButtonClick(event);
+        return;
+      }
+
+      if (this.autoOpenDisabled) {
         // Do not restore selected date if Esc was pressed after clearing input field
         if (this.inputElement.value === '') {
           this._selectDate(null);

package/src/vaadin-date-picker-overlay-content.js

@@ -235,7 +235,7 @@ class DatePickerOverlayContent extends ControllerMixin(ThemableMixin(DirMixin(Po
     return {
       scrollDuration: {
         type: Number,
-        default: 300,
+        value: 300,
       },
 
       /**

package/src/vaadin-date-picker.js

@@ -186,7 +186,7 @@ class DatePicker extends DatePickerMixin(InputControlMixin(ThemableMixin(Element
   }
 
   /**
-   * 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}
    */