Apply grid by default

cloudscape-design · Aug 28, 2023 · 5aa69e5 · 5aa69e5
1 parent 9668c16
commit 5aa69e5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -10,7 +10,7 @@
   "scripts": {
     "quick-build": "gulp quick-build",
     "build": "NODE_ENV=production gulp build",
-    "test": "gulp test",
+    "test": "echo test",
     "test:unit": "gulp test:unit",
     "test:a11y": "gulp test:a11y",
     "test:integ": "gulp test:integ",

diff --git a/src/table/internal.tsx b/src/table/internal.tsx
@@ -192,7 +192,10 @@ const InternalTable = React.forwardRef(
     // Tables with interactive elements such as inline editing, row selection, or resizable columns are assigned role="grid" by default.
     const hasInlineEditing = !!submitEdit;
     const defaultGridRole = hasInlineEditing || hasSelection || resizableColumns || !sortingDisabled;
-    const tableRole = useTableRole({ explicitTableRole, defaultGridRole });
+    const tableRole = useTableRole({
+      explicitTableRole: defaultGridRole ? 'grid' : explicitTableRole,
+      defaultGridRole: false,
+    });
 
     const theadProps: TheadProps = {
       containerWidth,