fix(b-table): prefer user-provided role attribute (#6382)

jacobmllr95 · web-flow · commit 9e25a3b97e91 · 2021-01-31T21:53:05.000+01:00
diff --git a/src/components/table/helpers/mixin-selectable.js b/src/components/table/helpers/mixin-selectable.js
@@ -70,13 +70,15 @@ export const selectableMixin = Vue.extend({
       }
     },
     selectableTableAttrs() {
+      const role = this.bvAttrs.role || 'grid'
+
       return this.isSelectable
         ? {
-            role: 'grid',
+            role,
             // TODO:
             //   Should this attribute not be included when `no-select-on-click` is set
             //   since this attribute implies keyboard navigation?
-            'aria-multiselectable': String(this.selectableIsMultiSelect)
+            'aria-multiselectable': role === 'grid' ? String(this.selectableIsMultiSelect) : null
           }
         : {}
     }

Original file line number	Diff line number	Diff line change
`@@ -70,13 +70,15 @@ export const selectableMixin = Vue.extend({`
`70`	`70`	`}`
`71`	`71`	`},`
`72`	`72`	`selectableTableAttrs() {`
	`73`	`+ const role = this.bvAttrs.role \|\| 'grid'`
	`74`	`+`
`73`	`75`	`return this.isSelectable`
`74`	`76`	`? {`
`75`		`- role: 'grid',`
	`77`	`+ role,`
`76`	`78`	`// TODO:`
`77`	`79`	// Should this attribute not be included when `no-select-on-click` is set
`78`	`80`	`// since this attribute implies keyboard navigation?`
`79`		`- 'aria-multiselectable': String(this.selectableIsMultiSelect)`
	`81`	`+ 'aria-multiselectable': role === 'grid' ? String(this.selectableIsMultiSelect) : null`
`80`	`82`	`}`
`81`	`83`	`: {}`
`82`	`84`	`}`