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

Revise the color of the page background on content pages to pass accessibility contrast test #107

Closed
gissoo opened this issue Mar 14, 2023 · 8 comments
Assignees
Labels
🗺️ design Tracks design work in an external app

Comments

@gissoo
Copy link
Contributor

gissoo commented Mar 14, 2023

@jhimpele @rlskoeser When designing the content pages our homepage background color that you currently see on the site is not passing the WCAG AAA test (for reading smaller text on the page, against our text color).
Therefore I had to revise our colors a little. Please look at the new colors on the content pageyou can also see what it would have looked like, which failed the accessibility test

I have two proposals, we can choose one from below:

  • We can revise the page background color everywhere on the site (including the homepage) so that it's a bit darker, and revise the header/footer/leaf panel colors too so that there is still a contrast between the two colors.
    Screenshot 2023-03-14 at 6.06.27 PM.png

  • Or we can only revise the page background color on the content pages (root pages) and leave the homepage as is. But we will still need to change the header/footer/leaf panel colors so that they match everywhere on the site.
    Screenshot 2023-03-14 at 6.06.49 PM.png

@gissoo gissoo self-assigned this Mar 14, 2023
@gissoo gissoo added 💬 awaiting review Ready for comments and questions 🗺️ design Tracks design work in an external app labels Mar 14, 2023
@jhimpele
Copy link
Contributor

@gissoo I'm sorry, I'm having a hard time following what is the old, and what are the new options. It looks like you are proposing a darker background for the content pages, which has less contrast with the header/footer. But we may or may not have to darken the main tree page background. Is that right? If so, I prefer to make the content pages dark, but keep the tree page lighter.... @rlskoeser what do you think.

@gissoo
Copy link
Contributor Author

gissoo commented Mar 21, 2023

@rlskoeser @jhimpele based on our discussions yesterday I have revised and have two proposals here on Figma

  • in proposal 1 I have a new background color we could apply to all pages that's warmer (based on your feedback), and I have also revised the header and footer and leaf panel colors to have more contrast against this background color. (and made the drop shadow of the panel starker too)
  • in proposal 2 is similar to proposal 1 but the only difference is I am keeping the homepage's background color as it was before.

Which one do you like more?
I just don't really like the amount of contrast between the two colors on the homepage in proposal 2.

I'd like to also not invert the colors as I am seeing it will need more adjustments to text styles for accessibility, and it conceptually doesn't make sense to me with the roots section being lighter than the body.

@rlskoeser
Copy link
Contributor

@gissoo thank you for finding a warmer color and looking at our options! I prefer proposal 2 but I'd be ok with proposal 1.

There's enough contrast for me on proposal 1, not sure the drop shadow is necessary.

@jhimpele
Copy link
Contributor

@gissoo thanks for working on this. I like proposal 1 because it makes all pages the same and I find the new contrast on the homepage header and footer too harsh. That said, is there any leeway to brighten the main area of the pages?

@gissoo
Copy link
Contributor Author

gissoo commented Mar 22, 2023

@jhimpele thanks for writing, by "...brighten the main area of the pages" do you mean make the new page background color slightly lighter?

@jhimpele
Copy link
Contributor

jhimpele commented Mar 22, 2023 via email

@gissoo
Copy link
Contributor Author

gissoo commented Mar 22, 2023

@jhimpele I think anything lighter than the new background color will be quite similar to what we had before –

@jhimpele @rlskoeser since neither of you feel strongly about this and are open about either option I propose we go with proposal 1 since it's the safest for accessibility, and since we like the glow for the plaque which would work on this darker background.

I'm also still not convinced that the drop shadow on the leaf labels on the lighter background is enough for accessibility, I don't really have a way to test it on my end.

@gissoo gissoo closed this as completed Mar 22, 2023
@gissoo gissoo removed the 💬 awaiting review Ready for comments and questions label Mar 22, 2023
@jhimpele
Copy link
Contributor

jhimpele commented Mar 22, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗺️ design Tracks design work in an external app
Projects
None yet
Development

No branches or pull requests

3 participants