From fcd4afa4fd4912cd1e5c55e554a2eec5765ffe12 Mon Sep 17 00:00:00 2001
From: DanielCliftonGuardian
 <110032454+DanielCliftonGuardian@users.noreply.github.com>
Date: Mon, 5 Jan 2026 14:32:49 +0000
Subject: [PATCH] Fix aria-prohibited-attr accessibility violations

---
 .../src/components/CarouselNavigationButtons.tsx             | 1 +
 dotcom-rendering/src/components/Contributor.tsx              | 1 -
 dotcom-rendering/src/components/Lineups.tsx                  | 5 +++--
 dotcom-rendering/src/components/SecureSignup.importable.tsx  | 2 +-
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/dotcom-rendering/src/components/CarouselNavigationButtons.tsx b/dotcom-rendering/src/components/CarouselNavigationButtons.tsx
index dcb8809eb87..2d509ea822b 100644
--- a/dotcom-rendering/src/components/CarouselNavigationButtons.tsx
+++ b/dotcom-rendering/src/components/CarouselNavigationButtons.tsx
@@ -86,6 +86,7 @@ export const CarouselNavigationButtons = ({
 
 	return ReactDOM.createPortal(
 		<div
+			role="group"
 			aria-controls="carousel"
 			aria-label="carousel arrows"
 			css={buttonStyles}
diff --git a/dotcom-rendering/src/components/Contributor.tsx b/dotcom-rendering/src/components/Contributor.tsx
index 3445bd0b0f7..39394a66beb 100644
--- a/dotcom-rendering/src/components/Contributor.tsx
+++ b/dotcom-rendering/src/components/Contributor.tsx
@@ -106,7 +106,6 @@ type Props = {
 
 export const Contributor = ({ byline, tags, format, source }: Props) => (
 	<address
-		aria-label="Contributor info"
 		data-component="meta-byline"
 		data-link-name="byline"
 		data-gu-name="byline"
diff --git a/dotcom-rendering/src/components/Lineups.tsx b/dotcom-rendering/src/components/Lineups.tsx
index 99ccd52f6f6..4aae1799a91 100644
--- a/dotcom-rendering/src/components/Lineups.tsx
+++ b/dotcom-rendering/src/components/Lineups.tsx
@@ -74,21 +74,22 @@ const Event = ({
 			return (
 				<span
 					css={rectangle(BackgroundRed)}
+					role="img"
 					aria-label="Red Card"
-					aria-hidden="false"
 				/>
 			);
 		case 'booking':
 			return (
 				<span
 					css={rectangle(BackgroundYellow)}
+					role="img"
 					aria-label="Yellow Card"
-					aria-hidden="false"
 				/>
 			);
 		case 'substitution':
 			return (
 				<span
+					role="img"
 					aria-label={`Substitution in ${time} minute`}
 					css={substitute}
 				>
diff --git a/dotcom-rendering/src/components/SecureSignup.importable.tsx b/dotcom-rendering/src/components/SecureSignup.importable.tsx
index c744ff6b210..8d1a94d8b61 100644
--- a/dotcom-rendering/src/components/SecureSignup.importable.tsx
+++ b/dotcom-rendering/src/components/SecureSignup.importable.tsx
@@ -444,7 +444,7 @@ export const SecureSignup = ({
 				</Button>
 			</form>
 			{isWaitingForResponse && (
-				<div aria-label="loading">
+				<div role="status" aria-label="loading">
 					<Spinner size="small" />
 				</div>
 			)}