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

Add label text to all form controls #2588

Merged
merged 3 commits into from
Nov 13, 2023

Conversation

ystros
Copy link
Contributor

@ystros ystros commented Nov 3, 2023

Previously, placeholder text has been used to represent the label for form controls. This allows visual users to tell what content to enter in specific form controls. However, screen readers generally expect an explicit label to be associated with form controls.

Rather redesign all pages to add <label> elements for each control, this adds the aria-label attribute to each form control. This should allow screen readers to properly announce controls as the page is read.

Previously, placeholder text has been used to represent the label for
form controls. This allows visual users to tell what content to enter in
specific form controls. However, screen readers generally expect an
explicit label to be associated with form controls.

Rather redesign all pages to add `<label>` elements for each control,
this adds the `aria-label` attribute to each form control. This should
allow screen readers to properly announce controls as the page is read.
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/186402859

The labels on this github issue will be updated when the story is started.

@strehle
Copy link
Member

strehle commented Nov 6, 2023

@Tallicia can you retrigger concourse ?

@Tallicia
Copy link
Contributor

Tallicia commented Nov 9, 2023

It looks like these need to be created with local braches in this repo.

Copy link
Member

@strehle strehle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@strehle strehle added this to the 76.26.0 milestone Nov 12, 2023
@strehle strehle merged commit 26c63ae into cloudfoundry:develop Nov 13, 2023
20 checks passed
@cf-gitbot cf-gitbot added delivered accepted Accepted the issue and removed delivered labels Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Accepted the issue
Projects
Development

Successfully merging this pull request may close these issues.

4 participants