Skip to content

Commit 27666fb

Browse files
Merge pull request #14407 from guardian/more-meta-targeting-attributes
2 parents ec4915b + ec4aba5 commit 27666fb

File tree

6 files changed

+11
-2
lines changed

6 files changed

+11
-2
lines changed

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,7 @@ export const CommentCount = ({ discussionApiUrl, shortUrlId }: Props) => {
9494
css={containerStyles}
9595
data-testid="comment-counts"
9696
value={`${long} comments on this article`}
97+
data-gu-name="comment-count"
9798
>
9899
<a
99100
href="#comments"

dotcom-rendering/src/components/Contributor.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,7 @@ export const Contributor = ({ byline, tags, format, source }: Props) => (
9393
aria-label="Contributor info"
9494
data-component="meta-byline"
9595
data-link-name="byline"
96+
data-gu-name="byline"
9697
>
9798
{format.design !== ArticleDesign.Interview && (
9899
<div

dotcom-rendering/src/components/Dateline.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,11 @@ export const Dateline = ({
5353
};
5454
if (secondaryDateline && !secondaryDateline.includes(primaryDateline)) {
5555
return (
56-
<details css={datelineStyles} style={mobileColour}>
56+
<details
57+
css={datelineStyles}
58+
style={mobileColour}
59+
data-gu-name="dateline"
60+
>
5761
<summary css={primaryStyles}>
5862
<span css={hoverUnderline}>{primaryDateline}</span>
5963
</summary>
@@ -62,7 +66,7 @@ export const Dateline = ({
6266
);
6367
}
6468
return (
65-
<div css={datelineStyles} style={mobileColour}>
69+
<div css={datelineStyles} style={mobileColour} data-gu-name="dateline">
6670
{primaryDateline}
6771
</div>
6872
);

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ export const FollowWrapper = ({ id, displayName }: Props) => {
172172
}
173173
}
174174
`}
175+
data-gu-name="follow"
175176
>
176177
{showFollowTagButton && (
177178
<FollowTagButton

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ export const LiveblogNotifications = ({ id, displayName }: Props) => {
9696
}
9797
}
9898
`}
99+
data-gu-name="liveblog-notifications"
99100
>
100101
<FollowNotificationsButton
101102
isFollowing={isFollowingNotifications ?? false}

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ export const CopyNativeShareButton = ({
126126
sharedButtonStyles(sizeXSmall),
127127
isLiveBlogMeta && liveBlogMobileMeta(isCopied),
128128
])}
129+
data-gu-name="share-button"
129130
>
130131
{isCopied ? 'Link copied' : 'Share'}
131132
</Button>

0 commit comments

Comments
 (0)