search

processing / processing-website

Repository for the processing.org website
https://processing.org
GNU General Public License v2.0
60 stars 90 forks source link

Low contrast between background and foreground colors (text) #441

Open shahnoor-khan opened 1 year ago

shahnoor-khan commented 1 year ago

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 :)

SableRaf commented 1 year ago

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

SableRaf commented 1 year ago

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

shahnoor-khan commented 1 year ago

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.