Skip to content

Commit

Permalink
Fix aria-allowed-role failure
Browse files Browse the repository at this point in the history
This was done on purpose in #304, but unclear if it still applies?
  • Loading branch information
domoscargin authored and owenatgov committed Sep 25, 2024
1 parent 3766443 commit 5963953
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions __tests__/accessibility-audit.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,7 @@ async function axe(page) {
'region',
'color-contrast-enhanced',
'aria-allowed-attr',
'target-size',
'aria-allowed-role'
'target-size'
])

.withTags([
Expand Down
2 changes: 1 addition & 1 deletion src/patterns/payment-card-details/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ Show logos for the cards you accept as icons so users can see whether their card
Use Issuer Identification Number (IIN) lookups to validate the card number as the user enters it. Once you’ve been able to identify the user’s card type, leave only the relevant logo highlighted and grey out the others.

<div class="app-video">
<video class="app-video__player" role="region" aria-labelledby="card-number-video-description" controls muted>
<video class="app-video__player" aria-labelledby="card-number-video-description" controls muted>
<source src="card-number.mp4" type="video/mp4">
</video>
<p class="app-video__description" id="card-number-video-description">
Expand Down

0 comments on commit 5963953

Please sign in to comment.