-
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
chore(docs): migrate react-chrome to storybook #883
Conversation
const backgroundColor = getColor(hue, 600, theme); | ||
const LIGHT_COLOR = 'white'; | ||
const Chrome = React.forwardRef<HTMLDivElement, IChromeProps>(({ hue, isFluid, ...props }, ref) => { | ||
const theme = useContext(ThemeContext); |
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.
The withTheme
usage confuses the Storybook story-source plugin. This aligns the Theme usage with newer components.
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.
@Austin94 does this mean we'll be subsequently dropping withTheme
throughout?
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 think it would be a good idea. Cleans up our prop-sheets throughout and also aligns us with eventual changes in styled-components
v6+
@@ -6,13 +6,14 @@ | |||
*/ | |||
|
|||
import React, { HTMLAttributes } from 'react'; | |||
import { StyledHeaderItemWrapper } from '../../styled'; | |||
import { StyledHeaderItemWrapper, IStyledBaseHeaderItemProps } from '../../styled'; |
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.
Moving to TS for the examples found we were missing some types for this component.
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.
The index.html
link is broken.
@jzempel I've updated the root link for |
Description
This PR migrates our
react-chrome
documentation to storybook. It is a 1-to-1 migration so no changes in docs/examples should be noticeable.Checklist
π design updates are Garden Designer approved (add thedesigner as a reviewer)
yarn start
)?bedrock
)πββοΈ includes new unit tests