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

Form field label is descriptive needs context in expectation #423

Closed
rvantonisse opened this issue Feb 15, 2019 · 2 comments
Closed

Form field label is descriptive needs context in expectation #423

rvantonisse opened this issue Feb 15, 2019 · 2 comments
Assignees
Labels
Agenda item Rule Update Use this label for an existing rule that is being updated Rule Use this label for a new rule that does not exist already

Comments

@rvantonisse
Copy link

While using the "Form field label is descriptive" rule to audit certain pages we stumbled onto situations where we would fail the SC because the label alone was not descriptive enough. However when looking at WCAG it is clear that we should not fail the SC because the labels where descriptive enough in context. We think the rule should be updated and that the expectation should explictly mention the fact that labels and their context together are expected to be descriptive of the purpose.

For instance a fieldset or heading above a part of the form can provide additional information about what should be filled in. For example the labels (name and street) below are not enough to determine the purpose of the field, but together with their context (the heading above it) it is enough to determine the context.

Shipping adress

Name
Street
etc.

Billing adress

Name
Street
etc.

@annethyme annethyme added March 31 deadline Rule Update Use this label for an existing rule that is being updated Rule Use this label for a new rule that does not exist already labels Mar 14, 2019
@annethyme annethyme self-assigned this Mar 19, 2019
annethyme pushed a commit that referenced this issue Mar 19, 2019
Handling comments that has come in through: #423
@annethyme
Copy link
Collaborator

Working on it here: #447.

I am not sure I completely agree, but let's hear what reviewers say

@annethyme
Copy link
Collaborator

Closing issue, since there is a pull request for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda item Rule Update Use this label for an existing rule that is being updated Rule Use this label for a new rule that does not exist already
Projects
None yet
Development

No branches or pull requests

3 participants