@vaadin/radio-group 24.2.0-alpha1 → 24.2.0-alpha10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/radio-group",
-  "version": "24.2.0-alpha1",
+  "version": "24.2.0-alpha10",
   "publishConfig": {
     "access": "public"
   },
@@ -38,21 +38,21 @@
   ],
   "dependencies": {
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/a11y-base": "24.2.0-alpha1",
-    "@vaadin/component-base": "24.2.0-alpha1",
-    "@vaadin/field-base": "24.2.0-alpha1",
-    "@vaadin/vaadin-lumo-styles": "24.2.0-alpha1",
-    "@vaadin/vaadin-material-styles": "24.2.0-alpha1",
-    "@vaadin/vaadin-themable-mixin": "24.2.0-alpha1"
+    "@vaadin/a11y-base": "24.2.0-alpha10",
+    "@vaadin/component-base": "24.2.0-alpha10",
+    "@vaadin/field-base": "24.2.0-alpha10",
+    "@vaadin/vaadin-lumo-styles": "24.2.0-alpha10",
+    "@vaadin/vaadin-material-styles": "24.2.0-alpha10",
+    "@vaadin/vaadin-themable-mixin": "24.2.0-alpha10"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/testing-helpers": "^0.4.2",
+    "@vaadin/testing-helpers": "^0.5.0",
     "sinon": "^13.0.2"
   },
   "web-types": [
     "web-types.json",
     "web-types.lit.json"
   ],
-  "gitHead": "0dbb118320203ab6c0c07450a3e718815367589f"
+  "gitHead": "ca16b5f88b00ae05fb6d7c7e9874525048e389f0"
 }

package/src/vaadin-radio-button.d.ts

@@ -16,8 +16,15 @@ import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mix
  */
 export type RadioButtonCheckedChangedEvent = CustomEvent<{ value: boolean }>;
 
+/**
+ * Fired when the `dirty` property changes.
+ */
+export type RadioButtonDirtyChangedEvent = CustomEvent<{ value: boolean }>;
+
 export interface RadioButtonCustomEventMap {
   'checked-changed': RadioButtonCheckedChangedEvent;
+
+  'dirty-changed': RadioButtonDirtyChangedEvent;
 }
 
 export interface RadioButtonEventMap extends HTMLElementEventMap, RadioButtonCustomEventMap {}
@@ -56,6 +63,7 @@ export interface RadioButtonEventMap extends HTMLElementEventMap, RadioButtonCus
  * See [Styling Components](https://vaadin.com/docs/latest/styling/styling-components) documentation.
  *
  * @fires {CustomEvent} checked-changed - Fired when the `checked` property changes.
+ * @fires {CustomEvent} dirty-changed - Fired when the `dirty` property changes.
  */
 declare class RadioButton extends LabelMixin(
   CheckedMixin(DelegateFocusMixin(ActiveMixin(ElementMixin(ThemableMixin(ControllerMixin(HTMLElement)))))),

package/src/vaadin-radio-button.js

@@ -48,6 +48,7 @@ import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mix
  * See [Styling Components](https://vaadin.com/docs/latest/styling/styling-components) documentation.
  *
  * @fires {CustomEvent} checked-changed - Fired when the `checked` property changes.
+ * @fires {CustomEvent} dirty-changed - Fired when the `dirty` property changes.
  *
  * @extends HTMLElement
  * @mixes ControllerMixin

package/src/vaadin-radio-group.d.ts

@@ -15,6 +15,11 @@ import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mix
  */
 export type RadioGroupInvalidChangedEvent = CustomEvent<{ value: boolean }>;
 
+/**
+ * Fired when the `dirty` property changes.
+ */
+export type RadioGroupDirtyChangedEvent = CustomEvent<{ value: boolean }>;
+
 /**
  * Fired when the `value` property changes.
  */
@@ -28,6 +33,8 @@ export type RadioGroupValidatedEvent = CustomEvent<{ valid: boolean }>;
 export interface RadioGroupCustomEventMap {
   'invalid-changed': RadioGroupInvalidChangedEvent;
 
+  'dirty-changed': RadioGroupDirtyChangedEvent;
+
   'value-changed': RadioGroupValueChangedEvent;
 
   validated: RadioGroupValidatedEvent;
@@ -74,6 +81,7 @@ export interface RadioGroupEventMap extends HTMLElementEventMap, RadioGroupCusto
  * See [Styling Components](https://vaadin.com/docs/latest/styling/styling-components) documentation.
  *
  * @fires {CustomEvent} invalid-changed - Fired when the `invalid` property changes.
+ * @fires {CustomEvent} dirty-changed - Fired when the `dirty` property changes.
  * @fires {CustomEvent} value-changed - Fired when the `value` property changes.
  * @fires {CustomEvent} validated - Fired whenever the field is validated.
  */
@@ -93,6 +101,15 @@ declare class RadioGroup extends FieldMixin(
    */
   readonly: boolean;
 
+  /**
+   * Whether the field is dirty.
+   *
+   * The field is automatically marked as dirty once the user triggers
+   * a `change` event. Additionally, the field can be manually marked
+   * as dirty by setting the property to `true`.
+   */
+  dirty: boolean;
+
   addEventListener<K extends keyof RadioGroupEventMap>(
     type: K,
     listener: (this: RadioGroup, ev: RadioGroupEventMap[K]) => void,

package/src/vaadin-radio-group.js

@@ -3,12 +3,12 @@
  * Copyright (c) 2017 - 2023 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { FlattenedNodesObserver } from '@polymer/polymer/lib/utils/flattened-nodes-observer.js';
 import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
 import { DisabledMixin } from '@vaadin/a11y-base/src/disabled-mixin.js';
 import { FocusMixin } from '@vaadin/a11y-base/src/focus-mixin.js';
 import { KeyboardMixin } from '@vaadin/a11y-base/src/keyboard-mixin.js';
 import { ElementMixin } from '@vaadin/component-base/src/element-mixin.js';
+import { SlotObserver } from '@vaadin/component-base/src/slot-observer.js';
 import { TooltipController } from '@vaadin/component-base/src/tooltip-controller.js';
 import { generateUniqueId } from '@vaadin/component-base/src/unique-id-utils.js';
 import { FieldMixin } from '@vaadin/field-base/src/field-mixin.js';
@@ -95,6 +95,11 @@ class RadioGroup extends FieldMixin(
           width: 100%;
         }
 
+        [part='group-field'] {
+          display: flex;
+          flex-wrap: wrap;
+        }
+
         :host(:not([has-label])) [part='label'] {
           display: none;
         }
@@ -158,6 +163,19 @@ class RadioGroup extends FieldMixin(
       _fieldName: {
         type: String,
       },
+
+      /**
+       * Whether the field is dirty.
+       *
+       * The field is automatically marked as dirty once the user triggers
+       * a `change` event. Additionally, the field can be manually marked
+       * as dirty by setting the property to `true`.
+       */
+      dirty: {
+        type: Boolean,
+        value: false,
+        notify: true,
+      },
     };
   }
 
@@ -166,6 +184,7 @@ class RadioGroup extends FieldMixin(
 
     this.__registerRadioButton = this.__registerRadioButton.bind(this);
     this.__unregisterRadioButton = this.__unregisterRadioButton.bind(this);
+    this.__onRadioButtonChange = this.__onRadioButtonChange.bind(this);
     this.__onRadioButtonCheckedChange = this.__onRadioButtonCheckedChange.bind(this);
   }
 
@@ -208,7 +227,8 @@ class RadioGroup extends FieldMixin(
 
     this._fieldName = `${this.localName}-${generateUniqueId()}`;
 
-    this._observer = new FlattenedNodesObserver(this, ({ addedNodes, removedNodes }) => {
+    const slot = this.shadowRoot.querySelector('slot:not([name])');
+    this._observer = new SlotObserver(slot, ({ addedNodes, removedNodes }) => {
       // Registers the added radio buttons in the reverse order
       // in order for the group to take the value of the most recent button.
       this.__filterRadioButtons(addedNodes).reverse().forEach(this.__registerRadioButton);
@@ -317,6 +337,7 @@ class RadioGroup extends FieldMixin(
    */
   __registerRadioButton(radioButton) {
     radioButton.name = this._fieldName;
+    radioButton.addEventListener('change', this.__onRadioButtonChange);
     radioButton.addEventListener('checked-changed', this.__onRadioButtonCheckedChange);
 
     if (this.disabled || this.readonly) {
@@ -335,6 +356,7 @@ class RadioGroup extends FieldMixin(
    * @private
    */
   __unregisterRadioButton(radioButton) {
+    radioButton.removeEventListener('change', this.__onRadioButtonChange);
     radioButton.removeEventListener('checked-changed', this.__onRadioButtonCheckedChange);
 
     if (radioButton.value === this.value) {
@@ -342,6 +364,11 @@ class RadioGroup extends FieldMixin(
     }
   }
 
+  /** @private */
+  __onRadioButtonChange() {
+    this.dirty = true;
+  }
+
   /**
    * @param {!CustomEvent} event
    * @private
@@ -459,7 +486,9 @@ class RadioGroup extends FieldMixin(
   _setFocused(focused) {
     super._setFocused(focused);
 
-    if (!focused) {
+    // Do not validate when focusout is caused by document
+    // losing focus, which happens on browser tab switch.
+    if (!focused && document.hasFocus()) {
       this.validate();
     }
   }

package/theme/lumo/vaadin-radio-group-styles.js

@@ -27,7 +27,6 @@ const radioGroup = css`
   }
 
   :host([theme~='vertical']) [part='group-field'] {
-    display: flex;
     flex-direction: column;
   }
 

package/theme/material/vaadin-radio-group-styles.js

@@ -27,7 +27,6 @@ const radioGroup = css`
   }
 
   :host([theme~='vertical']) [part='group-field'] {
-    display: flex;
     flex-direction: column;
   }
 

package/web-types.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/web-types",
   "name": "@vaadin/radio-group",
-  "version": "24.2.0-alpha1",
+  "version": "24.2.0-alpha10",
   "description-markup": "markdown",
   "contributions": {
     "html": {
@@ -32,6 +32,17 @@
                 ]
               }
             },
+            {
+              "name": "dirty",
+              "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\nan `input` or `change` event. Additionally, the field can be manually\nmarked as dirty by setting the property to `true`.",
+              "value": {
+                "type": [
+                  "boolean",
+                  "null",
+                  "undefined"
+                ]
+              }
+            },
             {
               "name": "checked",
               "description": "True if the element is checked.",
@@ -97,6 +108,17 @@
                   ]
                 }
               },
+              {
+                "name": "dirty",
+                "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\nan `input` or `change` event. Additionally, the field can be manually\nmarked as dirty by setting the property to `true`.",
+                "value": {
+                  "type": [
+                    "boolean",
+                    "null",
+                    "undefined"
+                  ]
+                }
+              },
               {
                 "name": "checked",
                 "description": "True if the element is checked.",
@@ -132,6 +154,10 @@
                 "name": "value-changed",
                 "description": "Fired when the `value` property changes."
               },
+              {
+                "name": "dirty-changed",
+                "description": "Fired when the `dirty` property changes."
+              },
               {
                 "name": "checked-changed",
                 "description": "Fired when the `checked` property changes."
@@ -249,6 +275,17 @@
                 ]
               }
             },
+            {
+              "name": "dirty",
+              "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\na `change` event. Additionally, the field can be manually marked\nas dirty by setting the property to `true`.",
+              "value": {
+                "type": [
+                  "boolean",
+                  "null",
+                  "undefined"
+                ]
+              }
+            },
             {
               "name": "theme",
               "description": "The theme variants to apply to the component.",
@@ -368,6 +405,17 @@
                     "boolean"
                   ]
                 }
+              },
+              {
+                "name": "dirty",
+                "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\na `change` event. Additionally, the field can be manually marked\nas dirty by setting the property to `true`.",
+                "value": {
+                  "type": [
+                    "boolean",
+                    "null",
+                    "undefined"
+                  ]
+                }
               }
             ],
             "events": [
@@ -379,6 +427,10 @@
                 "name": "value-changed",
                 "description": "Fired when the `value` property changes."
               },
+              {
+                "name": "dirty-changed",
+                "description": "Fired when the `dirty` property changes."
+              },
               {
                 "name": "invalid-changed",
                 "description": "Fired when the `invalid` property changes."

package/web-types.lit.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/web-types",
   "name": "@vaadin/radio-group",
-  "version": "24.2.0-alpha1",
+  "version": "24.2.0-alpha10",
   "description-markup": "markdown",
   "framework": "lit",
   "framework-config": {
@@ -26,6 +26,13 @@
                 "kind": "expression"
               }
             },
+            {
+              "name": "?dirty",
+              "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\nan `input` or `change` event. Additionally, the field can be manually\nmarked as dirty by setting the property to `true`.",
+              "value": {
+                "kind": "expression"
+              }
+            },
             {
               "name": "?checked",
               "description": "True if the element is checked.",
@@ -61,6 +68,13 @@
                 "kind": "expression"
               }
             },
+            {
+              "name": "@dirty-changed",
+              "description": "Fired when the `dirty` property changes.",
+              "value": {
+                "kind": "expression"
+              }
+            },
             {
               "name": "@checked-changed",
               "description": "Fired when the `checked` property changes.",
@@ -103,6 +117,13 @@
                 "kind": "expression"
               }
             },
+            {
+              "name": "?dirty",
+              "description": "Whether the field is dirty.\n\nThe field is automatically marked as dirty once the user triggers\na `change` event. Additionally, the field can be manually marked\nas dirty by setting the property to `true`.",
+              "value": {
+                "kind": "expression"
+              }
+            },
             {
               "name": ".label",
               "description": "The label text for the input node.\nWhen no light dom defined via [slot=label], this value will be used.",
@@ -159,6 +180,13 @@
                 "kind": "expression"
               }
             },
+            {
+              "name": "@dirty-changed",
+              "description": "Fired when the `dirty` property changes.",
+              "value": {
+                "kind": "expression"
+              }
+            },
             {
               "name": "@invalid-changed",
               "description": "Fired when the `invalid` property changes.",