From 4edf0afaeafd3d4ae3763fd3451a6154a64d261d Mon Sep 17 00:00:00 2001
From: Tomi Virkki <tomivirkki@users.noreply.github.com>
Date: Wed, 5 Jan 2022 15:16:56 +0200
Subject: [PATCH] fix: change grid table role to treegrid (#3259)

---
 packages/grid/src/vaadin-grid.js         | 2 +-
 packages/grid/test/accessibility.test.js | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/grid/src/vaadin-grid.js b/packages/grid/src/vaadin-grid.js
index 9221d9f869..e9b96ce7cb 100644
--- a/packages/grid/src/vaadin-grid.js
+++ b/packages/grid/src/vaadin-grid.js
@@ -279,7 +279,7 @@ class Grid extends ElementMixin(
         loading$="[[loading]]"
         column-reordering-allowed$="[[columnReorderingAllowed]]"
       >
-        <table id="table" role="grid" aria-multiselectable="true" tabindex="0">
+        <table id="table" role="treegrid" aria-multiselectable="true" tabindex="0">
           <caption id="sizer" part="row"></caption>
           <thead id="header" role="rowgroup"></thead>
           <tbody id="items" role="rowgroup"></tbody>
diff --git a/packages/grid/test/accessibility.test.js b/packages/grid/test/accessibility.test.js
index 64995c9b1a..9972d16705 100644
--- a/packages/grid/test/accessibility.test.js
+++ b/packages/grid/test/accessibility.test.js
@@ -111,8 +111,8 @@ describe('accessibility', () => {
     });
 
     describe('structural roles', () => {
-      it('should have role grid on table', () => {
-        expect(grid.$.table.getAttribute('role')).to.equal('grid');
+      it('should have role treegrid on table', () => {
+        expect(grid.$.table.getAttribute('role')).to.equal('treegrid');
       });
 
       it('should have role rowgroup on row groups', () => {