Skip to content

Conversation

@tomrf1
Copy link
Member

@tomrf1 tomrf1 commented Jan 29, 2026

Currently in PROD the pill is red because that's what SDC sets in the choice cards settings if there's a discount:
Screenshot 2026-01-29 at 15 34 10

The choice cards component "pill" colour can be configured in two ways:

  1. from the banner design tool
  2. from the choice card settings

Currently in the client the second overrides the first. But for banners we'd like to prioritise the banner design tool colours.
This PR changes the pill colours logic to do this.
For epics we'll still set the pill colour based on the choice card settings.

I've also added new storybook entries for the choice cards component which demonstrate this new logic

@github-actions
Copy link

github-actions bot commented Jan 29, 2026

@tomrf1 tomrf1 added the feature Departmental tracking: work on a new feature label Jan 29, 2026
}

export interface ChoiceCardSettings {
export interface ChoiceCardDesignSettings {
Copy link
Member Author

Choose a reason for hiding this comment

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

renamed for clarity - this comes from the banner design tool

@tomrf1 tomrf1 added the run_chromatic Runs chromatic when label is applied label Jan 29, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 29, 2026
@tomrf1 tomrf1 marked this pull request as ready for review January 29, 2026 15:54
@tomrf1 tomrf1 requested a review from a team as a code owner January 29, 2026 15:54
@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.

@tomrf1 tomrf1 added the run_chromatic Runs chromatic when label is applied label Jan 29, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 29, 2026
Copy link
Contributor

@AnastasiiaBalenko AnastasiiaBalenko left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

3 participants