Skip to content

Conversation

@oliverabrahams
Copy link
Contributor

@oliverabrahams oliverabrahams commented Nov 19, 2025

What does this change?

Moves the setting of the data component upstream to the cards.

For links rendered by the render element function (In body)

data-component='in-body-product-link'
data-link-name='product link button primary' | 'product link button tertiary'

For links rendered by the left-col-card

data-component='product-card-left-col'
data-link-name='product link button primary' | 'product link button tertiary'

For links rendered by the inline-product-card

data-component='product-card-type-inline' | 'product-card-type-inline-only'
data-link-name='product link button primary' | 'product link button tertiary'

For the product image links

data-component='product-card-type-inline' | 'product-card-type-inline-only' | 'product-card-left-col'
data-link-name='product image link'

@oliverabrahams oliverabrahams changed the title moving setting the data-component upstream Moving setting the data-component upstream Nov 19, 2025
@oliverabrahams oliverabrahams changed the title Moving setting the data-component upstream Product Link Buttons - move setting the data-component upstream Nov 19, 2025
@oliverabrahams oliverabrahams marked this pull request as ready for review November 20, 2025 14:29
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@github-actions
Copy link

github-actions bot commented Nov 20, 2025

@github-actions
Copy link

github-actions bot commented Nov 20, 2025

@oliverabrahams oliverabrahams added the run_chromatic Runs chromatic when label is applied label Nov 25, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Nov 25, 2025
@oliverabrahams oliverabrahams added the run_chromatic Runs chromatic when label is applied label Nov 25, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Nov 25, 2025
@oliverabrahams oliverabrahams merged commit 8358c06 into main Nov 25, 2025
32 of 33 checks passed
@oliverabrahams oliverabrahams deleted the oa/improve-tracking-take-3 branch November 25, 2025 10:10
@gu-prout
Copy link

gu-prout bot commented Nov 25, 2025

Seen on PROD (merged by @oliverabrahams 8 minutes and 15 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants