Skip to content

[CCR] Accessibility fixes#27268

Merged
cjcenizal merged 3 commits intoelastic:feature/ccrfrom
cjcenizal:ccr/accessibility
Dec 17, 2018
Merged

[CCR] Accessibility fixes#27268
cjcenizal merged 3 commits intoelastic:feature/ccrfrom
cjcenizal:ccr/accessibility

Conversation

@cjcenizal
Copy link
Contributor

@cjcenizal cjcenizal commented Dec 14, 2018

  • Add "Actions" title to actions column in CCR table so that it's clear what that column contains
  • Convert table column headers from nodes to strings so that VoiceOver doesn't read them as "Object object"
  • Fix bug where CCR table cells aren't being read by screen reader (edit: looks like this is a bug with VoiceOver and Chrome)

@cjcenizal cjcenizal added Feature:CCR and Remote Clusters Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// labels Dec 14, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@sebelga sebelga left a comment

Choose a reason for hiding this comment

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

LGTM!

@cjcenizal cjcenizal merged commit 2ccbb41 into elastic:feature/ccr Dec 17, 2018
@cjcenizal cjcenizal deleted the ccr/accessibility branch December 17, 2018 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:CCR and Remote Clusters Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t//

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants