@vaadin/dialog 25.3.0-alpha6 → 25.3.0-alpha8

package/custom-elements.json

@@ -796,7 +796,8 @@
             {
               "kind": "method",
               "name": "requestContentUpdate",
-              "description": "Requests an update for the content of the dialog.\nWhile performing the update, it invokes the renderer passed in the `renderer` property,\nas well as `headerRender` and `footerRenderer` properties, if these are defined.\n\nIt is not guaranteed that the update happens immediately (synchronously) after it is requested."
+              "description": "Requests an update for the content of the dialog.\nWhile performing the update, it invokes the renderer passed in the `renderer` property,\nas well as `headerRender` and `footerRenderer` properties, if these are defined.\n\nIt is not guaranteed that the update happens immediately (synchronously) after it is requested.",
+              "deprecated": "This method is only used with renderers and will be removed in Vaadin 26"
             }
           ],
           "attributes": [
@@ -1195,6 +1196,7 @@
               "kind": "method",
               "name": "requestContentUpdate",
               "description": "Requests an update for the content of the dialog.\nWhile performing the update, it invokes the renderer passed in the `renderer` property,\nas well as `headerRender` and `footerRenderer` properties, if these are defined.\n\nIt is not guaranteed that the update happens immediately (synchronously) after it is requested.",
+              "deprecated": "This method is only used with renderers and will be removed in Vaadin 26",
               "inheritedFrom": {
                 "name": "DialogRendererMixin",
                 "module": "src/vaadin-dialog-renderer-mixin.js"
@@ -1307,46 +1309,6 @@
               }
             }
           ],
-          "mixins": [
-            {
-              "name": "DialogSizeMixin",
-              "module": "src/vaadin-dialog-size-mixin.js"
-            },
-            {
-              "name": "DialogDraggableMixin",
-              "module": "src/vaadin-dialog-draggable-mixin.js"
-            },
-            {
-              "name": "DialogResizableMixin",
-              "module": "src/vaadin-dialog-resizable-mixin.js"
-            },
-            {
-              "name": "DialogRendererMixin",
-              "module": "src/vaadin-dialog-renderer-mixin.js"
-            },
-            {
-              "name": "DialogBaseMixin",
-              "module": "src/vaadin-dialog-base-mixin.js"
-            },
-            {
-              "name": "ThemePropertyMixin",
-              "package": "@vaadin/vaadin-themable-mixin/vaadin-theme-property-mixin.js"
-            },
-            {
-              "name": "ElementMixin",
-              "package": "@vaadin/component-base/src/element-mixin.js"
-            },
-            {
-              "name": "PolylitMixin",
-              "package": "@vaadin/component-base/src/polylit-mixin.js"
-            }
-          ],
-          "superclass": {
-            "name": "LitElement",
-            "package": "lit"
-          },
-          "tagName": "vaadin-dialog",
-          "customElement": true,
           "attributes": [
             {
               "name": "draggable",
@@ -1532,6 +1494,13 @@
                 "module": "src/vaadin-dialog-resizable-mixin.js"
               }
             },
+            {
+              "type": {
+                "text": "string"
+              },
+              "description": "The theme variants to apply to the component.",
+              "name": "theme"
+            },
             {
               "name": "top",
               "type": {
@@ -1556,7 +1525,47 @@
                 "module": "src/vaadin-dialog-size-mixin.js"
               }
             }
-          ]
+          ],
+          "mixins": [
+            {
+              "name": "DialogSizeMixin",
+              "module": "src/vaadin-dialog-size-mixin.js"
+            },
+            {
+              "name": "DialogDraggableMixin",
+              "module": "src/vaadin-dialog-draggable-mixin.js"
+            },
+            {
+              "name": "DialogResizableMixin",
+              "module": "src/vaadin-dialog-resizable-mixin.js"
+            },
+            {
+              "name": "DialogRendererMixin",
+              "module": "src/vaadin-dialog-renderer-mixin.js"
+            },
+            {
+              "name": "DialogBaseMixin",
+              "module": "src/vaadin-dialog-base-mixin.js"
+            },
+            {
+              "name": "ThemePropertyMixin",
+              "package": "@vaadin/vaadin-themable-mixin/vaadin-theme-property-mixin.js"
+            },
+            {
+              "name": "ElementMixin",
+              "package": "@vaadin/component-base/src/element-mixin.js"
+            },
+            {
+              "name": "PolylitMixin",
+              "package": "@vaadin/component-base/src/polylit-mixin.js"
+            }
+          ],
+          "superclass": {
+            "name": "LitElement",
+            "package": "lit"
+          },
+          "tagName": "vaadin-dialog",
+          "customElement": true
         }
       ],
       "exports": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/dialog",
-  "version": "25.3.0-alpha6",
+  "version": "25.3.0-alpha8",
   "publishConfig": {
     "access": "public"
   },
@@ -37,19 +37,19 @@
   ],
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.3.0",
-    "@vaadin/component-base": "25.3.0-alpha6",
-    "@vaadin/lit-renderer": "25.3.0-alpha6",
-    "@vaadin/overlay": "25.3.0-alpha6",
-    "@vaadin/vaadin-themable-mixin": "25.3.0-alpha6",
+    "@vaadin/component-base": "25.3.0-alpha8",
+    "@vaadin/lit-renderer": "25.3.0-alpha8",
+    "@vaadin/overlay": "25.3.0-alpha8",
+    "@vaadin/vaadin-themable-mixin": "25.3.0-alpha8",
     "lit": "^3.0.0"
   },
   "devDependencies": {
-    "@vaadin/a11y-base": "25.3.0-alpha6",
-    "@vaadin/aura": "25.3.0-alpha6",
-    "@vaadin/chai-plugins": "25.3.0-alpha6",
-    "@vaadin/test-runner-commands": "25.3.0-alpha6",
+    "@vaadin/a11y-base": "25.3.0-alpha8",
+    "@vaadin/aura": "25.3.0-alpha8",
+    "@vaadin/chai-plugins": "25.3.0-alpha8",
+    "@vaadin/test-runner-commands": "25.3.0-alpha8",
     "@vaadin/testing-helpers": "^2.0.0",
-    "@vaadin/vaadin-lumo-styles": "25.3.0-alpha6",
+    "@vaadin/vaadin-lumo-styles": "25.3.0-alpha8",
     "sinon": "^22.0.0"
   },
   "customElements": "custom-elements.json",
@@ -57,5 +57,5 @@
     "web-types.json",
     "web-types.lit.json"
   ],
-  "gitHead": "92c124fb9cff367bc07e734d8e65707facd0bd43"
+  "gitHead": "ccbb4aaffb63c745c6da0426b532d4d05e47af29"
 }

package/src/vaadin-dialog-base-mixin.js

@@ -198,10 +198,11 @@ export const DialogBaseMixin = (superClass) =>
     }
 
     /** @private */
-    _bringOverlayToFront() {
-      if (this.modeless) {
-        this._overlayElement.bringToFront();
+    _bringOverlayToFront(event) {
+      if (!this.modeless) {
+        return;
       }
+      this._overlayElement.bringToFront(event);
     }
 
     /** @private */

package/src/vaadin-dialog-overlay.js

@@ -7,6 +7,7 @@ import { html, LitElement } from 'lit';
 import { defineCustomElement } from '@vaadin/component-base/src/define.js';
 import { DirMixin } from '@vaadin/component-base/src/dir-mixin.js';
 import { PolylitMixin } from '@vaadin/component-base/src/polylit-mixin.js';
+import { getOverlaysOnTop, isNestedOverlay } from '@vaadin/overlay/src/vaadin-overlay-stack-mixin.js';
 import { LumoInjectionMixin } from '@vaadin/vaadin-themable-mixin/lumo-injection-mixin.js';
 import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 import { dialogOverlayStyles } from './styles/vaadin-dialog-overlay-base-styles.js';
@@ -15,6 +16,7 @@ import { DialogOverlayMixin } from './vaadin-dialog-overlay-mixin.js';
 /**
  * An element used internally by `<vaadin-dialog>`. Not intended to be used separately.
  *
+ * @attr {string} theme - The theme variants to apply to the component.
  * @customElement vaadin-dialog-overlay
  * @extends HTMLElement
  * @private
@@ -31,11 +33,11 @@ export class DialogOverlay extends DialogOverlayMixin(
   }
 
   /**
-   * Override method from OverlayFocusMixin to use owner as focus trap root
+   * Override method from OverlayFocusMixin to use owner as focus root
    * @protected
    * @override
    */
-  get _focusTrapRoot() {
+  get _focusRoot() {
     return this.owner;
   }
 
@@ -55,6 +57,29 @@ export class DialogOverlay extends DialogOverlayMixin(
       </div>
     `;
   }
+
+  /**
+   * @param {Event} event
+   * @protected
+   * @override
+   */
+  bringToFront(event) {
+    if (event instanceof Event) {
+      const path = event.composedPath();
+
+      // Do not bring overlay to front if this method is called
+      // on event that originates from a nested dialog content.
+      const isNestedDialogEvent = getOverlaysOnTop(this).some(
+        (overlay) => path.includes(overlay) && isNestedOverlay(this, overlay) && overlay instanceof DialogOverlay,
+      );
+
+      if (isNestedDialogEvent) {
+        return;
+      }
+    }
+
+    super.bringToFront();
+  }
 }
 
 defineCustomElement(DialogOverlay);

package/src/vaadin-dialog-renderer-mixin.d.ts

@@ -69,6 +69,8 @@ export declare class DialogRendererMixinClass {
    * as well as `headerRender` and `footerRenderer` properties, if these are defined.
    *
    * It is not guaranteed that the update happens immediately (synchronously) after it is requested.
+   *
+   * @deprecated This method is only used with renderers and will be removed in Vaadin 26
    */
   requestContentUpdate(): void;
 }

package/src/vaadin-dialog-renderer-mixin.js

@@ -77,6 +77,8 @@ export const DialogRendererMixin = (superClass) =>
      * as well as `headerRender` and `footerRenderer` properties, if these are defined.
      *
      * It is not guaranteed that the update happens immediately (synchronously) after it is requested.
+     *
+     * @deprecated This method is only used with renderers and will be removed in Vaadin 26
      */
     requestContentUpdate() {
       if (this._overlayElement) {

package/src/vaadin-dialog.js

@@ -116,6 +116,7 @@ export { DialogOverlay } from './vaadin-dialog-overlay.js';
  * @fires {CustomEvent} opened-changed - Fired when the `opened` property changes.
  * @fires {CustomEvent} closed - Fired when the dialog is closed.
  *
+ * @attr {string} theme - The theme variants to apply to the component.
  * @customElement vaadin-dialog
  * @extends HTMLElement
  */

package/web-types.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/web-types",
   "name": "@vaadin/dialog",
-  "version": "25.3.0-alpha6",
+  "version": "25.3.0-alpha8",
   "description-markup": "markdown",
   "contributions": {
     "html": {
@@ -123,9 +123,7 @@
               "description": "The theme variants to apply to the component.",
               "value": {
                 "type": [
-                  "string",
-                  "null",
-                  "undefined"
+                  "string"
                 ]
               }
             },

package/web-types.lit.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/web-types",
   "name": "@vaadin/dialog",
-  "version": "25.3.0-alpha6",
+  "version": "25.3.0-alpha8",
   "description-markup": "markdown",
   "framework": "lit",
   "framework-config": {