-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
#503 added useTheme hook documentation #569
#503 added useTheme hook documentation #569
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/styledcomponents/styled-components/CSeMkNP1bbbtUtdZWrkpauZAiyw2 |
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 is off to a great start @arthurfalcao! Thanks!
Can you please try your hand at adding a couple of words about the hook on the theming section too though? There is a paragraph about accessing the theme using the useContext
hook to get the theme, I think it could be integrated or even replaced by a mention of useTheme
.
/bump @arthurfalcao |
9757408
to
9399840
Compare
any help needed with this one? |
just needs to be reviewed |
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.
SGTM
The only nitpick that I have is that IMO useTheme
should come before useContext
in the docs but that's probably quite subjective. Do you have any further comments on this @probablyup ?
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.
Thank you!
Related issue: #503
Added useTheme hook in api helpers section