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

Add captions to user and space grid pages #82713

Merged
merged 4 commits into from
Nov 9, 2020

Conversation

legrego
Copy link
Member

@legrego legrego commented Nov 5, 2020

Summary

Adds table captions to the user and space grid pages to improve accessibly.

Resolves #43018
Resolves #42505

@legrego legrego added Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.11.0 labels Nov 5, 2020
@legrego legrego requested a review from a team as a code owner November 5, 2020 13:41
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@lizozom
Copy link
Contributor

lizozom commented Nov 5, 2020

@legrego
Copy link
Member Author

legrego commented Nov 5, 2020

@lizozom sorry I don't have access to the link you posted. Are there additional changes I need to make in this PR?

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-accessibility (Project:Accessibility)

Copy link
Contributor

@myasonik myasonik left a comment

Choose a reason for hiding this comment

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

The fix looks great, just a small nit on wording!
🎉 🎉 🎉

@legrego legrego requested a review from thomheymann November 5, 2020 20:07
@legrego
Copy link
Member Author

legrego commented Nov 9, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

async chunks size

id before after diff
security 814.3KB 814.5KB +157.0B

page load bundle size

id before after diff
spaces 290.2KB 290.4KB +168.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@thomheymann thomheymann left a comment

Choose a reason for hiding this comment

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

LGTM.

Caption comes through correctly (Users; Page 1 of 1. and Kibana spaces; Page 1 of 1.) and row headers (<th scope="row">) are set correctly for both tables.

@legrego legrego merged commit 48dbf9d into elastic:master Nov 9, 2020
@legrego legrego deleted the a11y/fix-user-spaces-grids branch November 9, 2020 19:16
legrego added a commit to legrego/kibana that referenced this pull request Nov 9, 2020
* Add captions to user and space grid pages

* Address PR feedback: reword captions

* remove unused i18n values

Co-authored-by: Kibana Machine <[email protected]>
legrego added a commit that referenced this pull request Nov 9, 2020
* Add captions to user and space grid pages

* Address PR feedback: reword captions

* remove unused i18n values

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
legrego added a commit that referenced this pull request Nov 9, 2020
* Add captions to user and space grid pages

* Address PR feedback: reword captions

* remove unused i18n values

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Kibana Machine <[email protected]>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Nov 10, 2020
* master: (39 commits)
  Fix ilm navigation (elastic#81664)
  [Lens] Distinct icons for XY and pie chart value labels toolbar (elastic#82927)
  [data.search.aggs] Throw an error when trying to create an agg type that doesn't exist. (elastic#81509)
  Index patterns api - load field list on server (elastic#82629)
  New events resolver (elastic#82170)
  [App Search] Misc naming tech debt (elastic#82770)
  load empty_kibana in test to have clean starting point (elastic#82772)
  Remove data <--> expressions circular dependencies. (elastic#82685)
  Update 8.0 breaking change template to gather information on how to programmatically detect it. (elastic#82905)
  Add alerting as codeowners to related documentation folder (elastic#82777)
  Add captions to user and space grid pages (elastic#82713)
  add alternate path for x-pack/Cloud test for Lens (elastic#82634)
  Uses asCurrentUser in getClusterUuid (elastic#82908)
  [Alerting][Connectors] Add new executor subaction to get 3rd party case fields (elastic#82519)
  Fix test import objects (elastic#82767)
  [ML] Add option for anomaly charts for metric detector should plot min, mean or max as appropriate (elastic#81662)
  Update alert type selection layout to rows instead of grid (elastic#73665)
  Prevent Kerberos and PKI providers from initiating a new session for unauthenticated XHR/API requests. (elastic#82817)
  Update grunt and related packages (elastic#79327)
  Allow the repository to search across all namespaces (elastic#82863)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! v7.10.0 v7.11.0 v8.0.0
Projects
None yet
6 participants