Merge pull request #675 from Financial-Times/CI-1367-dac-label-in-nam…

…e-10 fix aria-label from inputs
Financial-Times · Oct 27, 2022 · fb6796d · fb6796d
2 parents 00cee85 + 116e030
commit fb6796d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/components/x-gift-article/src/ReferAFriend.jsx b/components/x-gift-article/src/ReferAFriend.jsx
@@ -18,7 +18,7 @@ export default ({ rafTitle, rafDescription, urls, actions }) => {
 						value={urls.raf}
 						className="x-gift-article__url-input"
 						readOnly
-						aria-label="Gift free subscription shareable link"
+						aria-label={rafTitle}
 					/>
 				</span>
 				<div className="x-gift-article__buttons">

diff --git a/components/x-gift-article/src/Url.jsx b/components/x-gift-article/src/Url.jsx
@@ -1,7 +1,7 @@
 import { h } from '@financial-times/x-engine'
 import { ShareType } from './lib/constants'
 
-export default ({ shareType, isGiftUrlCreated, url, urlType }) => {
+export default ({ shareType, isGiftUrlCreated, url, urlType, ariaLabel }) => {
 	return (
 		<span className="o-forms-input o-forms-input--text">
 			<input
@@ -11,7 +11,7 @@ export default ({ shareType, isGiftUrlCreated, url, urlType }) => {
 				className="x-gift-article__url-input"
 				disabled={(shareType === ShareType.gift || shareType === ShareType.enterprise) && !isGiftUrlCreated}
 				readOnly
-				aria-label="Gift article shareable link"
+				aria-label={ariaLabel}
 			/>
 		</span>
 	)

diff --git a/components/x-gift-article/src/UrlSection.jsx b/components/x-gift-article/src/UrlSection.jsx
@@ -23,7 +23,8 @@ export default ({
 	enterpriseLimit,
 	enterpriseHasCredits,
 	enterpriseRequestAccess,
-	enterpriseFirstTimeUser
+	enterpriseFirstTimeUser,
+	title
 }) => {
 	const hideUrlShareElements =
 		(giftCredits === 0 && shareType === ShareType.gift) ||
@@ -42,7 +43,8 @@ export default ({
 						shareType,
 						isGiftUrlCreated,
 						url,
-						urlType
+						urlType,
+						ariaLabel: title
 					}}
 				/>
 			)}