civictechindex / CTI-website-frontend

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

Properly label form elements for accessibility #1102

Closed bruceplai closed 2 years ago

bruceplai commented 2 years ago

Overview

Form elements such as input fields are not currently labeled properly. They won't be scanned by site accessibility readers.

Action Items

Resources/Instructions

image

mealthebear commented 2 years ago

Progress: I haven't been able to start this issue over the weekend. Blockers: None identified so far Availability: I'll have some days this week to work on this issue ETA: I'm thinking for next Thursday (2/10/2022)

mealthebear commented 2 years ago

Progress: Spent some time researching what might be the best way to add labels to inputs. It seems like aria-labels are a good option since they are intended for assistive technologies. Blockers: None so far. Availability: I'll be able to work over the weekend on this issue. ETA: 2/10/2022 should be on schedule

mealthebear commented 2 years ago

Progress: I haven't been able to work this week due to other responsibilities Blockers: Material UI components renders a series of nested elements, so passing an attribute to the intended element is not as straight forward as raw HTML for example. Will require some research and reading. Availability: I'll have a little time over the weekend for this issue. ETA: 2/24/2022 Another week should be enough time to figure out the problem

mealthebear commented 2 years ago

1162

mealthebear commented 2 years ago

Progress: I haven't made any updates over the weekend Blockers: I'm still researching how to pass attributes to some of the pre-existing Material UI components that were built Availability: I'll have some time tomorrow to work on this issue ETA: 3/10/22

smsada commented 2 years ago

A couple of nice to haves like a couple extra tests remain but functionality is there so issue is closed