Skip to content

Conversation

@emma-imber
Copy link
Contributor

@emma-imber emma-imber commented Jan 7, 2026

What does this change?

Adds a new StackedCards component, using the Horizontal Product Summary Card.

Why?

As another variant for our At a Glance A/B/C test.

Screenshots

Breakpoint Light mode Dark mode
Mobile Screenshot 2026-01-12 at 17 37 45 Screenshot 2026-01-12 at 17 37 53
Desktop Screenshot 2026-01-07 at 17 30 07 Screenshot 2026-01-07 at 17 30 16

@emma-imber emma-imber added the run_chromatic Runs chromatic when label is applied label Jan 7, 2026
@emma-imber emma-imber changed the title Ei/add stacked horizontal cards Add stacked horizontal product cards Jan 7, 2026
@emma-imber emma-imber changed the title Add stacked horizontal product cards Add stacked horizontal product cards component Jan 7, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 7, 2026
@emma-imber emma-imber added the feature Departmental tracking: work on a new feature label Jan 9, 2026
@github-actions
Copy link

github-actions bot commented Jan 12, 2026

@emma-imber emma-imber marked this pull request as ready for review January 13, 2026 10:24
@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.

@emma-imber emma-imber added the run_chromatic Runs chromatic when label is applied label Jan 13, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 13, 2026
<div css={informationContainer}>
<div
css={productCardHeading}
dangerouslySetInnerHTML={{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How come we were able to remove this? Is it to do with the extractHeadingText?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep exactly! We're able to do this because we're not passing in html any more, because of using extractHeadingText.

Copy link
Contributor

@charleycampbell charleycampbell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Just one question but it's just out of curiosity than a blocker!

`,
]}
>
<button
Copy link
Contributor

@oliverabrahams oliverabrahams Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know we haven't used the DS for the read more using the link but we should look into changing that

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will have a look at using it!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also could we add tracking to these clicks

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

Labels

feature Departmental tracking: work on a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants