Skip to content

Conversation

@juabara
Copy link
Contributor

@juabara juabara commented Oct 29, 2025

What does this change?

Set unselected choice card colour.

@juabara juabara requested a review from a team as a code owner October 29, 2025 16:14
@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 Oct 29, 2025

@juabara juabara changed the title fix: attempt to set the unselected choice card text colour straight set the unselected choice card text colour straight Oct 29, 2025
color: ${isSelected
? choiceCardSettings?.buttonSelectTextColour ?? 'inherit'
: choiceCardSettings?.buttonTextColour ?? 'inherit'};
s {
Copy link
Contributor

Choose a reason for hiding this comment

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

What does the s stand for on this line?

Copy link
Contributor Author

@juabara juabara Oct 29, 2025

Choose a reason for hiding this comment

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

Before the buttonTextColour was setting the colour to the label, but the inner css was overring the text colour, so I moved the label override to the label div and applied the buttonTextColour to it. The s was already there, so I prefered to keep it.

Copy link
Member

@tomrf1 tomrf1 Oct 30, 2025

Choose a reason for hiding this comment

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

it's applying style to strikethrough tags (<s>).
This is used when a promo is present and it displays the old price in <s> tags

@juabara juabara added the run_chromatic Runs chromatic when label is applied label Oct 29, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Oct 29, 2025
@juabara juabara merged commit d124a5a into main Oct 30, 2025
31 checks passed
@juabara juabara deleted the jm/banner-colour-palettes branch October 30, 2025 08:46
@gu-prout
Copy link

gu-prout bot commented Oct 30, 2025

Seen on PROD (merged by @juabara 8 minutes and 12 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.

4 participants