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

(Accessibility) Kibna Logs - Multiple "gear" icons share same ID #40414

Closed
barlowm opened this issue Jul 5, 2019 · 4 comments
Closed

(Accessibility) Kibna Logs - Multiple "gear" icons share same ID #40414

barlowm opened this issue Jul 5, 2019 · 4 comments
Labels
Feature:Logs UI Logs UI feature Project:Accessibility Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services WCAG A

Comments

@barlowm
Copy link
Collaborator

barlowm commented Jul 5, 2019

Use Deque axe Browser Extension

Multiple "Gear" icons share the same ID

<path id="gear-a" ...>

id attribute value must be unique

Steps to reproduce

  1. Navigate to the Kibana Logs page
  2. Run aXe scan

Untitled6

Meta Issue
Accessibility Audit for Kibana 7.0

Kibana Version:
7.2

OS:
Tested on Windows 7. But issue exists in all OS's

Browser:
Tested in Chrome. But issue exists in all Browsers

Screen reader: [if relevant]
N/A

Relevant WCAG Criteria: WCAG Criterion
4.1.1 Parsing - Level A
In content implemented using markup languages, elements have complete start and end tags,
elements are nested according to their specifications, elements do not contain duplicate attributes, and any IDs are unique, except where the specifications allow these features.

@sgrodzicki sgrodzicki added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Jul 11, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-logs-ui

@Kerry350
Copy link
Contributor

These gear icons are rendered by the EuiButtonIcon component, with an iconType of gear. As the <path /> element is part of the actual SVG we'll need some help from the EUI team fixing this one. Pinging @elastic/eui-design.

@cchaos
Copy link
Contributor

cchaos commented Jul 22, 2019

We already have an issue in EUI to clean up all the SVG code: elastic/eui#1989

@Kerry350 Kerry350 removed their assignment Aug 6, 2019
@sgrodzicki
Copy link

This has been fixed in elastic/eui#2169

@zube zube bot reopened this Oct 29, 2019
@zube zube bot closed this as completed Oct 29, 2019
@zube zube bot reopened this Oct 29, 2019
@zube zube bot closed this as completed Oct 29, 2019
@zube zube bot reopened this Oct 29, 2019
@zube zube bot removed the [zube]: Done label Oct 29, 2019
@zube zube bot closed this as completed Oct 29, 2019
@zube zube bot reopened this Oct 29, 2019
@zube zube bot closed this as completed Oct 29, 2019
@zube zube bot reopened this Oct 29, 2019
@zube zube bot closed this as completed Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Project:Accessibility Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services WCAG A
Projects
None yet
Development

No branches or pull requests

5 participants