-
Notifications
You must be signed in to change notification settings - Fork 29
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
Ticker Update #12153
base: main
Are you sure you want to change the base?
Ticker Update #12153
Conversation
trying to get this to run like an animation
Storybook displaying new ticker
adjusted the padding and some other styles
Size Change: -1.18 kB (-0.12%) Total Size: 952 kB
ℹ️ View Unchanged
|
@@ -264,12 +264,15 @@ const DesignableBanner: ReactComponent<BannerRenderProps> = ({ | |||
imageSettings, | |||
bannerId: 'designable-banner', | |||
tickerStylingSettings: { | |||
textColour: hexColourToString(ticker.text), | |||
//text: hexColourToString(ticker.text), |
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.
some tidy up to do here
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.
nice catch - complete oversight by me!!
totalColour: '#5056F5', | ||
goalColour: '#000000', | ||
//text: '#000000', // Deprecated, still needs a value | ||
//goalMarker: '#000000', // Deprecated, still needs a value |
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.
can be removed
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.
nice catch - complete oversight by me!!
@@ -1,12 +1,7 @@ | |||
/** |
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.
can this file be removed?
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.
yes good spot it's not being used anymore
@@ -105,7 +105,7 @@ const MetaGridByline = ({ | |||
padding-bottom: 0px; | |||
} | |||
address > div > span > svg:first-of-type { | |||
address > div > span > div:first-of-type { |
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.
I don't think this change is related to the ticker is 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.
oh sorry, just noticed @domlander pushed this as a fix
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.
This change isn't related to the ticker, but is required for the update to source-development-kitchen
. I thought we could get this PR out with this small fix, but it turns out we can't update this without also updating source
at the same time, which will take a bit more effort.
What does this change?
Update the banner and Epic ticker to align with the most recent design in Source. Figma deisgn
Please note that the Epic ticker colours are currently hard-coded and not configurable in the RRCP. However, the banner ticker colours can be customised via the RRCP, as this is the existing behaviour.
This has been tested end to end on CODE by creating a new banner design and epic test and ensuring that both tests render on a DCR article.
Screenshots
New ticker:
On Epic:
On banner: