-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Chakra UI setup #7195
Chakra UI setup #7195
Conversation
Gatsby Cloud Build Reportethereum-org-website-dev 🎉 Your build was successful! See the Deploy preview here. Build Details🕐 Build time: 33m |
956c758
to
b0a1ab6
Compare
@@ -0,0 +1,12 @@ | |||
const styles = { | |||
global: (_props) => ({ | |||
// TODO: remove these overrides as we adopt the new Design System |
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.
A bit more documentation/context on why this is needed to begin with would be helpful.
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.
e.g. why is both this file & GlobalStyle.tsx needed?
const config: ThemeConfig = { | ||
cssVarPrefix: "eth", | ||
initialColorMode: "light", | ||
useSystemColorMode: false, |
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 we should use the system color mode personally, I enjoy this feature when I use websites.
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.
Agreed. I've changed 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.
LGTM. Tried to break a bunch of pages with no success yet.
Note: branched off from #7179 (depends on
emotion
)Description
Initial Chakra UI setup.
Things addressed in this PR:
GlobalStyle
componentRelated Issue
Epic #6374
Fixes #7196