fix: change grid table role to treegrid (#3259)

vaadin · Jan 5, 2022 · 4edf0af · 4edf0af
1 parent e9a57ba
commit 4edf0af
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git 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
@@ -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', () => {