Skip to content

Commit

Permalink
fix: use null instead of false to remove aria-disabled
Browse files Browse the repository at this point in the history
  • Loading branch information
davidnixon committed Nov 6, 2023
1 parent 641b828 commit a071aae
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/components/CvComboBox/CvComboBox.vue
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@
:aria-controls="cvId"
aria-autocomplete="list"
role="combobox"
:aria-disabled="disabled"
:aria-disabled="disabled || null"
:aria-expanded="open ? 'true' : 'false'"
autocomplete="off"
:disabled="disabled"
Expand Down
4 changes: 2 additions & 2 deletions src/components/CvDropdown/CvDropdown.vue
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@
<button
ref="button"
:aria-controls="`${uid}-menu`"
:aria-disabled="disabled"
:aria-disabled="disabled || null"
:aria-expanded="open ? 'true' : 'false'"
:aria-labelledby="ariaLabeledBy"
:class="`${carbonPrefix}--list-box__field`"
Expand Down Expand Up @@ -129,7 +129,7 @@
</div>
<div
v-else-if="data.isHelper"
:aria-disabled="disabled"
:aria-disabled="disabled || null"
:class="[
`${carbonPrefix}--form__helper-text`,
{ [`${carbonPrefix}--form__helper-text--disabled`]: disabled },
Expand Down
2 changes: 1 addition & 1 deletion src/components/CvMultiSelect/CvMultiSelect.vue
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
ref="elButton"
type="button"
:class="`${carbonPrefix}--list-box__field`"
:aria-disabled="disabled"
:aria-disabled="disabled || null"
aria-haspopup="listbox"
:aria-expanded="data.open ? 'true' : 'false'"
:aria-owns="uid"
Expand Down
2 changes: 1 addition & 1 deletion src/components/CvProgress/CvProgressStep.vue
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
`cv-progress-step ${carbonPrefix}--progress-step`,
`${carbonPrefix}--progress-step--${internalState}`,
]"
:aria-disabled="disabled"
:aria-disabled="disabled || null"
>
<button
type="button"
Expand Down
2 changes: 1 addition & 1 deletion src/components/CvTabs/CvTabs.vue
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@
:class="`${carbonPrefix}--tabs--scrollable__nav-link`"
role="tab"
:aria-controls="tab.uid"
:aria-disabled="disabledTabs.has(tab.uid)"
:aria-disabled="disabledTabs.has(tab.uid) || null"
:aria-selected="selectedId === tab.uid"
:tabindex="selectedId === tab.uid ? 0 : -1"
type="button"
Expand Down

0 comments on commit a071aae

Please sign in to comment.