-
Notifications
You must be signed in to change notification settings - Fork 30
set the unselected choice card text colour straight #14770
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Hello 👋! When you're ready to run Chromatic, please apply the You will need to reapply the label each time you want to run Chromatic. |
| color: ${isSelected | ||
| ? choiceCardSettings?.buttonSelectTextColour ?? 'inherit' | ||
| : choiceCardSettings?.buttonTextColour ?? 'inherit'}; | ||
| s { |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
What does this change?
Set unselected choice card colour.