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

a11y: missing labels on controls (text fields, select boxes) #6512

Open
VladimirAus opened this issue Oct 13, 2020 · 4 comments
Open

a11y: missing labels on controls (text fields, select boxes) #6512

VladimirAus opened this issue Oct 13, 2020 · 4 comments

Comments

@VladimirAus
Copy link

Swagger UI renderer breaks accessibility rules: WCAG 1.3.1, WCAG 4.1.2

  • path: http://localhost:1723
  • also text fields, checkboxes etc

Fix ONE of the following:

  • aria-label attribute does not exist or is empty
  • aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
  • Form element does not have an implicit (wrapped)
  • Form element does not have an explicit
  • Element has no title attribute or the title attribute is empty

See screenshot: fields in RED are failing

image

@tim-lai
Copy link
Contributor

tim-lai commented Oct 19, 2020

related: #5298

@Sundava
Copy link

Sundava commented Sep 27, 2021

I want to pick this one up, is that all right ?

If I understand correctly, an acceptable fix would be to make sure our forms have an unique aria-label attribute associated with the parameter, right ?
Reading the discussion #5298, I feel that something along the lines of aria-label=path method in parameter_name would fit the bill, are you okay with this ?
(in refering to body, path,query, as the path+method+name is not enough to ensure uniqueness)

@char0n char0n assigned char0n and unassigned char0n Sep 30, 2021
@msftedad
Copy link

Hi Team, is there any update on this bug? Is this issue fixed?

@ho823
Copy link

ho823 commented Oct 3, 2023

Hi :) Do you need any help for this issue ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants