We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
There was an error while loading. Please reload this page.
role="grid"
1 parent 0a14828 commit bc02fb8Copy full SHA for bc02fb8
src/components/table/helpers/mixin-selectable.js
@@ -70,16 +70,15 @@ export const selectableMixin = Vue.extend({
70
}
71
},
72
selectableTableAttrs() {
73
- return {
74
- // TODO:
75
- // Should this attribute not be included when no-select-on-click is set
76
- // since this attribute implies keyboard navigation?
77
- 'aria-multiselectable': !this.isSelectable
78
- ? null
79
- : this.selectableIsMultiSelect
80
- ? 'true'
81
- : 'false'
82
- }
+ return this.isSelectable
+ ? {
+ role: 'grid',
+ // 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)
+ }
+ : {}
83
84
85
watch: {
0 commit comments