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.

Sign up for GitHub

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

Jump to bottom

Monitor background color different from the rest of Weave Cloud #167

Closed
bia opened this issue Apr 9, 2018 · 0 comments
Closed

Monitor background color different from the rest of Weave Cloud #167

bia opened this issue Apr 9, 2018 · 0 comments
Assignees
@bia
Labels
style-guide Part of the effort towards a unified design system

Comments

@bia
Copy link
Contributor

bia commented Apr 9, 2018 •
edited
Loading

Elaborating from #93, now that we are measuring legibility in a quantitative way, WCAG 2 AA Compliance.

While most of the app's background color is #fafafc, Explore is #f8f8f8 lightgray and Monitor is #ffffff with #f8f8f8 lightgray overlayed blocks.
Both backgrounds should be changed to #fafafc for consistency.

If we do shift the Monitor background to be consistent with the rest of the app, we might consider shifting lightgray to some darker variant of #fafafc with the same hue. For example, #ededf0 (#fafafc with 5% less black).

Related to #65
cc @miklosp
@bia bia assigned dlespiau and fbarl Apr 9, 2018
@bia bia mentioned this issue Apr 10, 2018
Closed
@bia bia assigned bia and unassigned fbarl Apr 18, 2018
@fbarl fbarl added the style-guide Part of the effort towards a unified design system label Apr 18, 2018
@bia bia closed this as completed May 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bia bia

Labels
style-guide Part of the effort towards a unified design system
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dlespiau @bia @fbarl