Skip to content
This repository was archived by the owner on Mar 27, 2025. It is now read-only.

Commit fefe8fa

Browse files
authored
Merge pull request bootstrap-vue-next#1069 from ndavenia/main
fix: add accessibility to b-dropdown and reset show class
2 parents a3cb0e8 + 13d663e commit fefe8fa

File tree

1 file changed

+6
-3
lines changed
  • packages/bootstrap-vue-next/src/components/BDropdown

1 file changed

+6
-3
lines changed

packages/bootstrap-vue-next/src/components/BDropdown/BDropdown.vue

+6-3
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,10 @@
2222
:variant="variant"
2323
:size="size"
2424
:disabled="disabled"
25-
:class="toggleClass"
25+
:class="[toggleClass, ...[modelValueBoolean ? 'show' : undefined]]"
2626
class="dropdown-toggle-split dropdown-toggle"
27-
aria-expanded="false"
27+
:aria-expanded="modelValueBoolean"
28+
aria-haspopup="menu"
2829
@click="onButtonClick"
2930
>
3031
<span class="visually-hidden">
@@ -230,6 +231,7 @@ const buttonClasses = computed(() => [
230231
'dropdown-toggle': !splitBoolean.value,
231232
'dropdown-toggle-no-caret': noCaretBoolean.value && !splitBoolean.value,
232233
'w-100': splitBoolean.value && blockBoolean.value,
234+
'show': splitBoolean.value ? undefined : modelValueBoolean.value,
233235
},
234236
])
235237
@@ -241,7 +243,8 @@ const dropdownMenuClasses = computed(() => [
241243
])
242244
243245
const buttonAttr = computed(() => ({
244-
'aria-expanded': splitBoolean.value ? undefined : false,
246+
'aria-expanded': splitBoolean.value ? undefined : modelValueBoolean.value,
247+
'aria-haspopup': splitBoolean.value ? undefined :'menu',
245248
'href': splitBoolean.value ? props.splitHref : undefined,
246249
'to': splitBoolean.value && props.splitTo ? props.splitTo : undefined,
247250
}))

0 commit comments

Comments
 (0)