alphapapa / solarized-everything-css

A collection of Solarized user-stylesheets for...everything?
GNU General Public License v3.0
278 stars 43 forks source link

make check color consistent #48

Closed vcfvct closed 3 years ago

vcfvct commented 5 years ago

check and the surrounding circle have the same color. Changing it to black to keep consistent.

before

check-before

after

check-after

Please let me know if you have any concerns. Thanks.

jgkamat commented 5 years ago

diff --git a/sites/github.styl b/sites/github.styl index fe16019..17bafa6 100644 --- a/sites/github.styl +++ b/sites/github.styl @@ -649,7 +649,7 @@ a.label .octicon-x color red .octicon-check

  • color green
  • color black

I think octicon-check is used for the CI checks as well, so we can't do something like this or the CI checks (and checks used in other places) will become black.

.octicon-primitive-dot color yellow

vcfvct commented 5 years ago

@jgkamat , good point, so any suggest color to fix this? shall we jus NOT color it?

jgkamat commented 5 years ago

@jgkamat , good point, so any suggest color to fix this?

I'm not actually sure. we could look deeper to see how exactly github styles the two checks differently, since there must be a difference in their CSS, or we could dim the color a little bit on our end so the green isn't exactly the same as a hack :P.

Of course, the first solution is ideal, but requires a little bit of digging to figure out what's going on on github's side.

The-Compiler commented 5 years ago

Maybe style the div instead, e.g. the .completeness-indicator class?

alphapapa commented 5 years ago

Thanks for looking into this. Small suggestion: black is probably not the best color to use, as it clashes with the rest of the theme. We should probably use the default text color, as long as it has enough contrast. If not, it could be darkened slightly with a Stylus filter.