processing-website icon indicating copy to clipboard operation
processing-website copied to clipboard

Low contrast between background and foreground colors (text)

Open shahnoor-khan opened this issue 2 years ago • 4 comments

worked on issue #427 . changed the color contrast between background and hover color in footer, so the people with color blindness can also read clearly. Before the hover color of anchor tag used to be like this: image to this: Screenshot_20230213_062854

Changes made on footer: Screenshot_20230212_084104 to this: Screenshot_20230212_083818 I'm new to open source contributions. Can I work further on this issue #427 to improve the website? Any advice will be appreciated :)

shahnoor-khan avatar Feb 13 '23 13:02 shahnoor-khan

Thanks for your contribution @shahnoor-khan! ⭐️ Tagging @runemadsen for review

SableRaf avatar Feb 13 '23 13:02 SableRaf

Hi @shahnoor-khan,

Thanks for your work on issue #427. A couple of things before we can merge the PR:

  1. Could we change the yellow in the footer to a light blue that's already used in the site's design?
  2. While the on-click text color for cards is improved, the default grey could still use more contrast against the white.

Thanks for your help on this!

Raphaël

SableRaf avatar May 30 '23 14:05 SableRaf

Hey @SableRaf Thanks for reviewing changes.

  1. Yes, we could do that and I actually tried yellow because light and dark were getting mixed with each other but I could change that again.
  2. Ok, I'll go with darker gray there.

shahnoor-khan avatar Jun 02 '23 20:06 shahnoor-khan

@Stefterv Thanks for reviewing this but I think you may have missed the changes I requested in my May 30, 2023 reply :)

SableRaf avatar Jul 30 '24 14:07 SableRaf