-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add button to toggle theme in header #2980
Conversation
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 seems to work nicely! 💯 Next out is making the dark theme actually functional 😆
app/components/Header/index.js
Outdated
<ToggleTheme | ||
loggedIn={loggedIn} | ||
updateUserTheme={this.props.updateUserTheme} | ||
username={this.props.currentUser?.username} | ||
/> |
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.
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 agree, but I think it should be in the user drop-down or something on mobile, so it's not too hard to get to.
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 have added it to the dropdown now (when logged in)
8e8469b
to
98031f5
Compare
98031f5
to
3e36959
Compare
The button is an icon that switches based on the current theme. If the user is logged in, the theme will also be persisted to the backend.
3e36959
to
06d4310
Compare
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.
Looks awesome now 💯
The button is an icon that switches based on the current theme. If the
user is logged in, the theme will also be persisted to the
backend.