From ff30a54ebf5d5cc4cd415cd775593e8b8f8d6533 Mon Sep 17 00:00:00 2001
From: Sergey Vinogradov <mr.vursen@gmail.com>
Date: Thu, 11 Apr 2024 10:05:56 +0300
Subject: [PATCH] refactor: set role attribute directly in template (#7318)

---
 packages/dialog/src/vaadin-dialog.js     | 3 +--
 packages/dialog/src/vaadin-lit-dialog.js | 8 +-------
 2 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/packages/dialog/src/vaadin-dialog.js b/packages/dialog/src/vaadin-dialog.js
index eb57a8bca0f..49fe81d0342 100644
--- a/packages/dialog/src/vaadin-dialog.js
+++ b/packages/dialog/src/vaadin-dialog.js
@@ -103,6 +103,7 @@ class Dialog extends DialogDraggableMixin(
 
       <vaadin-dialog-overlay
         id="overlay"
+        role="dialog"
         header-title="[[headerTitle]]"
         on-opened-changed="_onOverlayOpened"
         on-mousedown="_bringOverlayToFront"
@@ -147,8 +148,6 @@ class Dialog extends DialogDraggableMixin(
   ready() {
     super.ready();
 
-    this._overlayElement.setAttribute('role', 'dialog');
-
     processTemplates(this);
   }
 
diff --git a/packages/dialog/src/vaadin-lit-dialog.js b/packages/dialog/src/vaadin-lit-dialog.js
index ba0e6a1dc72..77c3f79d852 100644
--- a/packages/dialog/src/vaadin-lit-dialog.js
+++ b/packages/dialog/src/vaadin-lit-dialog.js
@@ -72,6 +72,7 @@ class Dialog extends DialogDraggableMixin(
     return html`
       <vaadin-dialog-overlay
         id="overlay"
+        role="dialog"
         .owner="${this}"
         .opened="${this.opened}"
         .headerTitle="${this.headerTitle}"
@@ -91,13 +92,6 @@ class Dialog extends DialogDraggableMixin(
       ></vaadin-dialog-overlay>
     `;
   }
-
-  /** @protected */
-  ready() {
-    super.ready();
-
-    this._overlayElement.setAttribute('role', 'dialog');
-  }
 }
 
 defineCustomElement(Dialog);