-
Notifications
You must be signed in to change notification settings - Fork 49
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
fixed style3 inactive color #60
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 will probably be a problem in other styles too but it will be reworked in #42 anyway... The only thing is that we should conform to the documentation which uses the secondary color for the text and defaults to the primary color:
Co-authored-by: Jannis Berndt <berndtjannis@gmail.com>
Co-authored-by: Jannis Berndt <berndtjannis@gmail.com>
Oops, good catch on the edits. I didn't see the other PR. Should I check out the other styles too or not worry about it since it's getting updating in #42? |
Codecov Report
@@ Coverage Diff @@
## master #60 +/- ##
==========================================
- Coverage 73.86% 73.81% -0.06%
==========================================
Files 36 36
Lines 3069 3074 +5
==========================================
+ Hits 2267 2269 +2
- Misses 802 805 +3
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
I will fix them in #42, thanks for contributing :) |
style 3 was not correctly applying inactive color