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
Typography/Heading: add Cambio options & styles #323
Typography/Heading: add Cambio options & styles #323
Conversation
🦋 Changeset detectedLatest commit: 6c6ad56 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
08ed3f7
to
18aa111
Compare
18aa111
to
63899e4
Compare
63899e4
to
e64df9e
Compare
e64df9e
to
c4a83fa
Compare
@@ -21,28 +21,43 @@ export default { | |||
control: { type: "radio" }, | |||
}, | |||
color: { | |||
// | "gray900" |
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.
remove comments
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.
LGTM!
// | "gray900" | ||
// | "gray700" | ||
// | "primary" | ||
// | "destructive-primary" | ||
// | "destructive-darkBackground" | ||
// | "success" | ||
// | "success-darkBackground" | ||
// | "white" | ||
// | "inherit", |
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.
is this comment block intentional? If so, is it meant to maintain documentation on classic colors?
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.
@zprong removed - thank you
c4a83fa
to
6c6ad56
Compare
Background
Follow up to #317 which added a
ThemeProvider
and initial Cambio button styles. In this PR we add Cambio styles forTypography
&Heading
Changes
elevation
when Cambio theme is selectedVideo
cambio-typography-heading.mp4