@vaadin/select 23.0.9 → 23.0.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/select",
-  "version": "23.0.9",
+  "version": "23.0.12",
   "publishConfig": {
     "access": "public"
   },
@@ -34,24 +34,24 @@
   ],
   "dependencies": {
     "@polymer/polymer": "^3.2.0",
-    "@vaadin/button": "^23.0.9",
-    "@vaadin/component-base": "^23.0.9",
-    "@vaadin/field-base": "^23.0.9",
-    "@vaadin/input-container": "^23.0.9",
-    "@vaadin/item": "^23.0.9",
-    "@vaadin/list-box": "^23.0.9",
-    "@vaadin/vaadin-list-mixin": "^23.0.9",
-    "@vaadin/vaadin-lumo-styles": "^23.0.9",
-    "@vaadin/vaadin-material-styles": "^23.0.9",
-    "@vaadin/vaadin-overlay": "^23.0.9",
-    "@vaadin/vaadin-themable-mixin": "^23.0.9"
+    "@vaadin/button": "^23.0.12",
+    "@vaadin/component-base": "^23.0.12",
+    "@vaadin/field-base": "^23.0.12",
+    "@vaadin/input-container": "^23.0.12",
+    "@vaadin/item": "^23.0.12",
+    "@vaadin/list-box": "^23.0.12",
+    "@vaadin/vaadin-list-mixin": "^23.0.12",
+    "@vaadin/vaadin-lumo-styles": "^23.0.12",
+    "@vaadin/vaadin-material-styles": "^23.0.12",
+    "@vaadin/vaadin-overlay": "^23.0.12",
+    "@vaadin/vaadin-themable-mixin": "^23.0.12"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "^23.0.9",
+    "@vaadin/polymer-legacy-adapter": "^23.0.12",
     "@vaadin/testing-helpers": "^0.3.2",
     "lit": "^2.0.0",
     "sinon": "^9.2.0"
   },
-  "gitHead": "4d687bdd48ba78d55f3371a78b70818e4dfca1a3"
+  "gitHead": "717908c222c1e241259e30b4144cd5ba32734819"
 }

package/src/vaadin-select-overlay.js

@@ -15,7 +15,7 @@ registerStyles(
       justify-content: flex-start;
     }
   `,
-  { moduleId: 'vaadin-select-overlay-styles' }
+  { moduleId: 'vaadin-select-overlay-styles' },
 );
 
 /**

package/src/vaadin-select-value-button.js

@@ -38,7 +38,7 @@ registerStyles(
       line-height: inherit;
     }
   `,
-  { moduleId: 'vaadin-select-value-button-styles' }
+  { moduleId: 'vaadin-select-value-button-styles' },
 );
 
 /**

package/src/vaadin-select.d.ts

@@ -6,6 +6,7 @@
 import { ElementMixin } from '@vaadin/component-base/src/element-mixin.js';
 import { SlotMixin } from '@vaadin/component-base/src/slot-mixin.js';
 import { DelegateFocusMixin } from '@vaadin/field-base/src/delegate-focus-mixin.js';
+import { DelegateStateMixin } from '@vaadin/field-base/src/delegate-state-mixin.js';
 import { FieldMixin } from '@vaadin/field-base/src/field-mixin.js';
 import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 
@@ -164,7 +165,9 @@ export interface SelectEventMap extends HTMLElementEventMap, SelectCustomEventMa
  * @fires {CustomEvent} opened-changed - Fired when the `opened` property changes.
  * @fires {CustomEvent} value-changed - Fired when the `value` property changes.
  */
-declare class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(ThemableMixin(HTMLElement))))) {
+declare class Select extends DelegateFocusMixin(
+  DelegateStateMixin(FieldMixin(SlotMixin(ElementMixin(ThemableMixin(HTMLElement))))),
+) {
   /**
    * An array containing items that will be rendered as the options of the select.
    *
@@ -255,13 +258,13 @@ declare class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixi
   addEventListener<K extends keyof SelectEventMap>(
     type: K,
     listener: (this: Select, ev: SelectEventMap[K]) => void,
-    options?: boolean | AddEventListenerOptions
+    options?: boolean | AddEventListenerOptions,
   ): void;
 
   removeEventListener<K extends keyof SelectEventMap>(
     type: K,
     listener: (this: Select, ev: SelectEventMap[K]) => void,
-    options?: boolean | EventListenerOptions
+    options?: boolean | EventListenerOptions,
   ): void;
 }
 

package/src/vaadin-select.js

@@ -14,6 +14,7 @@ import { MediaQueryController } from '@vaadin/component-base/src/media-query-con
 import { SlotMixin } from '@vaadin/component-base/src/slot-mixin.js';
 import { processTemplates } from '@vaadin/component-base/src/templates.js';
 import { DelegateFocusMixin } from '@vaadin/field-base/src/delegate-focus-mixin.js';
+import { DelegateStateMixin } from '@vaadin/field-base/src/delegate-state-mixin.js';
 import { FieldMixin } from '@vaadin/field-base/src/field-mixin.js';
 import { fieldShared } from '@vaadin/field-base/src/styles/field-shared-styles.js';
 import { inputFieldContainer } from '@vaadin/field-base/src/styles/input-field-container-styles.js';
@@ -131,8 +132,11 @@ registerStyles('vaadin-select', [fieldShared, inputFieldContainer], { moduleId:
  * @mixes SlotMixin
  * @mixes FieldMixin
  * @mixes DelegateFocusMixin
+ * @mixes DelegateStateMixin
  */
-class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(ThemableMixin(PolymerElement))))) {
+class Select extends DelegateFocusMixin(
+  DelegateStateMixin(FieldMixin(SlotMixin(ElementMixin(ThemableMixin(PolymerElement))))),
+) {
   static get is() {
     return 'vaadin-select';
   }
@@ -209,7 +213,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
        */
       items: {
         type: Array,
-        observer: '__itemsChanged'
+        observer: '__itemsChanged',
       },
 
       /**
@@ -221,7 +225,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
         value: false,
         notify: true,
         reflectToAttribute: true,
-        observer: '_openedChanged'
+        observer: '_openedChanged',
       },
 
       /**
@@ -253,14 +257,14 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
         type: String,
         value: '',
         notify: true,
-        observer: '_valueChanged'
+        observer: '_valueChanged',
       },
 
       /**
        * The name of this element.
        */
       name: {
-        type: String
+        type: String,
       },
 
       /**
@@ -271,7 +275,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
        * DOM content is empty.
        */
       placeholder: {
-        type: String
+        type: String,
       },
 
       /**
@@ -281,7 +285,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
       readonly: {
         type: Boolean,
         value: false,
-        reflectToAttribute: true
+        reflectToAttribute: true,
       },
 
       /** @private */
@@ -289,7 +293,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
 
       /** @private */
       _phoneMediaQuery: {
-        value: '(max-width: 420px), (max-height: 420px)'
+        value: '(max-width: 420px), (max-height: 420px)',
       },
 
       /** @private */
@@ -299,16 +303,19 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
       _inputContainer: Object,
 
       /** @private */
-      _items: Object
+      _items: Object,
     };
   }
 
+  static get delegateAttrs() {
+    return [...super.delegateAttrs, 'invalid'];
+  }
+
   static get observers() {
     return [
       '_updateAriaExpanded(opened)',
-      '_updateAriaRequired(required)',
       '_updateSelectedItem(value, _items, placeholder)',
-      '_rendererChanged(renderer, _overlayElement)'
+      '_rendererChanged(renderer, _overlayElement)',
     ];
   }
 
@@ -320,7 +327,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
         const button = document.createElement('vaadin-select-value-button');
         button.setAttribute('aria-haspopup', 'listbox');
         return button;
-      }
+      },
     };
   }
 
@@ -346,11 +353,11 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
     if (this._valueButton) {
       this._valueButton.setAttribute('aria-labelledby', `${this._labelId} ${this._fieldId}`);
 
-      this._updateAriaRequired(this.required);
       this._updateAriaExpanded(this.opened);
 
       this._setFocusElement(this._valueButton);
       this.ariaTarget = this._valueButton;
+      this.stateTarget = this._valueButton;
 
       this._valueButton.addEventListener('keydown', this._boundOnKeyDown);
     }
@@ -376,7 +383,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
     this.addController(
       new MediaQueryController(this._phoneMediaQuery, (matches) => {
         this._phone = matches;
-      })
+      }),
     );
 
     processTemplates(this);
@@ -451,7 +458,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
           this.__userInteraction = true;
           this.opened = false;
         },
-        true
+        true,
       );
 
       menuElement.setAttribute('role', 'listbox');
@@ -485,7 +492,11 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
    *
    * @private
    */
-  _onClick() {
+  _onClick(event) {
+    // Prevent parent components such as `vaadin-grid`
+    // from handling the click event after it bubbles.
+    event.preventDefault();
+
     this.opened = !this.readonly;
   }
 
@@ -536,7 +547,7 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
 
       this._overlayElement.style.setProperty(
         '--vaadin-select-text-field-width',
-        this._inputContainer.offsetWidth + 'px'
+        this._inputContainer.offsetWidth + 'px',
       );
 
       // Preserve focus-ring to restore it later
@@ -565,13 +576,6 @@ class Select extends DelegateFocusMixin(FieldMixin(SlotMixin(ElementMixin(Themab
     }
   }
 
-  /** @private */
-  _updateAriaRequired(required) {
-    if (this._valueButton) {
-      this._valueButton.setAttribute('aria-required', required ? 'true' : 'false');
-    }
-  }
-
   /** @private */
   _updateAriaLive(ariaLive) {
     if (this._valueButton) {

package/theme/lumo/vaadin-select-styles.js

@@ -82,7 +82,7 @@ registerStyles(
       background-color: transparent;
     }
   `,
-  { moduleId: 'lumo-select-value-button' }
+  { moduleId: 'lumo-select-value-button' },
 );
 
 const selectOverlay = css`

package/theme/lumo/vaadin-select.js

@@ -7,5 +7,6 @@ import '@vaadin/button/theme/lumo/vaadin-button.js';
 import '@vaadin/input-container/theme/lumo/vaadin-input-container.js';
 import '@vaadin/item/theme/lumo/vaadin-item.js';
 import '@vaadin/list-box/theme/lumo/vaadin-list-box.js';
+import '@vaadin/vaadin-overlay/theme/lumo/vaadin-overlay.js';
 import './vaadin-select-styles.js';
 import '../../src/vaadin-select.js';

package/theme/material/vaadin-select-styles.js

@@ -63,7 +63,7 @@ registerStyles(
       background-color: transparent;
     }
   `,
-  { moduleId: 'material-select-value-button' }
+  { moduleId: 'material-select-value-button' },
 );
 
 const selectOverlay = css`

package/theme/material/vaadin-select.js

@@ -7,5 +7,6 @@ import '@vaadin/button/theme/material/vaadin-button.js';
 import '@vaadin/input-container/theme/material/vaadin-input-container.js';
 import '@vaadin/item/theme/material/vaadin-item.js';
 import '@vaadin/list-box/theme/material/vaadin-list-box.js';
+import '@vaadin/vaadin-overlay/theme/material/vaadin-overlay.js';
 import './vaadin-select-styles.js';
 import '../../src/vaadin-select.js';