Skip to content

Commit fcd4afa

Browse files
Fix aria-prohibited-attr accessibility violations
1 parent 4c25252 commit fcd4afa

File tree

4 files changed

+5
-4
lines changed

4 files changed

+5
-4
lines changed

dotcom-rendering/src/components/CarouselNavigationButtons.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ export const CarouselNavigationButtons = ({
8686

8787
return ReactDOM.createPortal(
8888
<div
89+
role="group"
8990
aria-controls="carousel"
9091
aria-label="carousel arrows"
9192
css={buttonStyles}

dotcom-rendering/src/components/Contributor.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,6 @@ type Props = {
106106

107107
export const Contributor = ({ byline, tags, format, source }: Props) => (
108108
<address
109-
aria-label="Contributor info"
110109
data-component="meta-byline"
111110
data-link-name="byline"
112111
data-gu-name="byline"

dotcom-rendering/src/components/Lineups.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,21 +74,22 @@ const Event = ({
7474
return (
7575
<span
7676
css={rectangle(BackgroundRed)}
77+
role="img"
7778
aria-label="Red Card"
78-
aria-hidden="false"
7979
/>
8080
);
8181
case 'booking':
8282
return (
8383
<span
8484
css={rectangle(BackgroundYellow)}
85+
role="img"
8586
aria-label="Yellow Card"
86-
aria-hidden="false"
8787
/>
8888
);
8989
case 'substitution':
9090
return (
9191
<span
92+
role="img"
9293
aria-label={`Substitution in ${time} minute`}
9394
css={substitute}
9495
>

dotcom-rendering/src/components/SecureSignup.importable.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ export const SecureSignup = ({
444444
</Button>
445445
</form>
446446
{isWaitingForResponse && (
447-
<div aria-label="loading">
447+
<div role="status" aria-label="loading">
448448
<Spinner size="small" />
449449
</div>
450450
)}

0 commit comments

Comments
 (0)