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.

Sign up for GitHub

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

Jump to bottom

Multiple label elements is not widely supported in assistive technologies violation on EUI switch #2416

Closed
bhavyarm opened this issue Oct 10, 2019 · 1 comment · Fixed by #2411
Closed

Multiple label elements is not widely supported in assistive technologies violation on EUI switch #2416

bhavyarm opened this issue Oct 10, 2019 · 1 comment · Fixed by #2411
Labels
accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility bug

Comments

@bhavyarm
Copy link
Contributor

While adding accessibility automation test here elastic/kibana#43584 we got this violation on EUI switch

  [form-field-multiple-labels]: Ensures form field does not have multiple label elements
    Help: https://dequeuniversity.com/rules/axe/3.3/form-field-multiple-labels?application=axeAPI
    Elements:
      - #x34sa6nn
      - #l5wkjpuj
      - #\32 7ykxrdg
      - #xf4h0hui
      - #tyj61w1q
      - #\38 rvp9ij2
      - #vtx6expn
      - #w1argemc
      - #isumc2cy
      - #\37 4ej5lec
      - #v0jhq85c
      - #c7bhr8ge
      - #\38 89224v6
      - #g0uq2k98
      - #n0sfxcx6
      - #av4dfybe
      - #wksofhbu
      - #fvf7hnxe

Screen Shot 2019-10-10 at 5 44 58 PM

Screen Shot 2019-10-10 at 5 45 10 PM
@bhavyarm bhavyarm added bug accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility labels Oct 10, 2019
@bhavyarm
Copy link
Contributor Author

Will be fixed by #2411

@myasonik myasonik mentioned this issue Oct 11, 2019
Merged
5 tasks
snide pushed a commit that referenced this issue Oct 11, 2019
@snide
Fixes several a11y issues manifesting in Kibana (#2411)
f475ce9 
Fixes several of misc a11y issues showing up in Kibana:
- Poor labeling in `EuiSuperDatePicker` ([Kibana issue 38794](elastic/kibana#38794)) (this kind of snowballed into improving several things in the vicinity)
- Fixes #2416: `EuiFormRow`s that render fields that also render a label end up rendering multiple labels for an input which breaks many assistive tech (this issue was found during the in progress Kibana rollout of automated a11y testing ([Kibana issue 43584](elastic/kibana#43584) )
- Fixes #2414: `EuiCodeEditor`s would render with the same ID which breaks many assistive tech (this issue was found during the in progress Kibana rollout of automated a11y testing)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility - WCAG AA Level AA WCAG Accessibility Criteria accessibility bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes several a11y issues manifesting in Kibana myasonik/eui
1 participant
@bhavyarm