Skip to content
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

Higher contrast color scheme #344

Closed
dcramer opened this issue Apr 7, 2020 · 6 comments · Fixed by #349
Closed

Higher contrast color scheme #344

dcramer opened this issue Apr 7, 2020 · 6 comments · Fixed by #349

Comments

@dcramer
Copy link
Contributor

dcramer commented Apr 7, 2020

It'd be great if the color scheme could have better contrast. A light blue background with white text is quite difficult to read.

Happy to @ some designers if thats valuable :)

@zhouzhuojie
Copy link
Collaborator

Yes please! I'm thinking of rewriting / redesign the UI for a while... not a frontend guy, can definitely use some help here.

@dcramer
Copy link
Contributor Author

dcramer commented Apr 7, 2020

@ckj maybe you could provide some thoughts?

@dcramer
Copy link
Contributor Author

dcramer commented Apr 7, 2020

Also @zhouzhuojie have a color/palette of choice? Can probably whip up a PR with a proposal.

@ckj
Copy link

ckj commented Apr 7, 2020

@dcramer @zhouzhuojie It looks like the light blue is a brand choice. What if you made the text darker? This one uses the color from the body copy on your website instead of the green var.

flagr_homepage

If you're not partial to the light blue brand-wise, the blue you're using for the active table sort icon is really nice:

flagr_app_blue

@wildefires
Copy link

On a similar note, I've gotten some requests internally for a 'dark theme' because developers always seem to love their dark themes

@zhouzhuojie
Copy link
Collaborator

LGTM, the blue can also be the dark blue from https://checkr.com/ :)
And yes, dark theme is awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants