search

civictechindex / CTI-website-frontend

Join a worldwide movement to catalog 
every open source 
civic tech project.
https://civictechindex.org
MIT License
31 stars 29 forks source link

Update link colors to pass accessibility test #1122

Closed bruceplai closed 2 years ago

bruceplai commented 2 years ago

Overview

Currently contrast for links is insufficient to pass accessibility scans. Colors should be updated to fix that.

Action Items

bruceplai commented 2 years ago

@chalimar can we use the same default link color as the button background update (#0A7A9E) ? Also different states of links use different colors, but different states of buttons use opacity. Should we switch links to use opacity as well, or come up with a new set of colors for different states? By states, I mean default, active, hover, visited. Thanks

https://www.w3schools.com/css/css_link.asp

EDIT: Or perhaps just updating the default color is enough to pass the accessibility tests.

bruceplai commented 2 years ago

@chalimar I had to use a darker shade of blue to pass the accessibility contrast tests. This new color will apply to most of the links and buttons (#0A75AD)

bruceplai commented 2 years ago

Link color updates not properly taking effect in brave browser (based on chrome) during my testing. If this issue can be repeated by other team members during multiple browser testing, then I will investigate further.