-
Notifications
You must be signed in to change notification settings - Fork 352
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
feat(example): Add dark theme toggle button & change background color #1212
Conversation
66b1ff8
to
a87f288
Compare
I removed the background color after discussing with @mcoker . The gray color was throwing off the styles for multiple components. I added a border on the right of the left nav to make sure there still a visual separation between the nav and main. |
@ibolton336 you are the man! |
PatternFly-React preview: https://1212-pr-patternfly-react-patternfly.surge.sh |
Pull Request Test Coverage Report for Build 4049
💛 - Coveralls |
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
a87f288
to
07241f7
Compare
@@ -0,0 +1,11 @@ | |||
html, |
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 file should be removed
07241f7
to
ab25844
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.
Nice! I actually removed it in the Tabs PR too :)
They were not displaying correctly otherwise.
What:
closes #957
Additional issues: