-
Notifications
You must be signed in to change notification settings - Fork 91
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
fix(theming): update palette color values #1795
Conversation
<Canvas> | ||
<Story | ||
name="PALETTE" | ||
args={{ palette: PALETTE }} |
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.
Should we let the user interact with the arguments for this story? If so, the current implementation will only track changes to the outer color keys. That's because Hue
gets the colors from the theme.palette and not the palette object injected as an argument.
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'm not super concerned about whether or not the story is interactive β more interested in demonstrating where the values are coming from. However, there is some copy/paste jank in here from the website
implementation. Now cleaned up. π§Ή
<Grid.Col sm> | ||
<Hue hue={hue1} /> | ||
</Grid.Col> | ||
{hue1 !== 'product' && <StyledCol sm>{hue2 && <Hue hue={hue2} />}</StyledCol>} |
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.
That's always true
since product
is excluded.
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.
Colors for the win! π π€©
Description
Updated v9 palette color values and added an associated story demo.
Checklist
npm start
)renders as expected with reversed (RTL) directionrenders as expected with Bedrock CSS (?bedrock
)tested in Chrome, Firefox, Safari, and Edge