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

[Search][a11y] Fix table row screen reader error #203136

Conversation

navarone-feekery
Copy link
Contributor

@navarone-feekery navarone-feekery commented Dec 5, 2024

Closes #199113

Allows errors in the table row to be read by screen readers.

Screen.Recording.2024-12-05.at.17.16.01.mov

@navarone-feekery navarone-feekery added release_note:skip Skip the PR/issue when compiling release notes Team:Search backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development labels Dec 5, 2024
@navarone-feekery navarone-feekery requested a review from a team as a code owner December 5, 2024 16:19
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 2.6MB 2.6MB +34.0B

@navarone-feekery navarone-feekery merged commit 1a9893b into elastic:main Dec 6, 2024
15 checks passed
@navarone-feekery navarone-feekery deleted the fix-screen-reading-table-errors branch December 6, 2024 09:10
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.15, 8.16, 8.17, 8.x

https://github.com/elastic/kibana/actions/runs/12196060124

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 6, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.

(cherry picked from commit 1a9893b)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 6, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.

(cherry picked from commit 1a9893b)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 6, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.

(cherry picked from commit 1a9893b)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 6, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.

(cherry picked from commit 1a9893b)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15
8.16
8.17
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 6, 2024
…3219)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Search][a11y] Fix table row screen reader error
(#203136)](#203136)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Navarone
Feekery","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-06T09:10:26Z","message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-major"],"title":"[Search][a11y]
Fix table row screen reader
error","number":203136,"url":"https://github.com/elastic/kibana/pull/203136","mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203136","number":203136,"mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}}]}]
BACKPORT-->

Co-authored-by: Navarone Feekery <[email protected]>
kibanamachine added a commit that referenced this pull request Dec 6, 2024
…03217)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Search][a11y] Fix table row screen reader error
(#203136)](#203136)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Navarone
Feekery","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-06T09:10:26Z","message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-major"],"title":"[Search][a11y]
Fix table row screen reader
error","number":203136,"url":"https://github.com/elastic/kibana/pull/203136","mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203136","number":203136,"mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}}]}]
BACKPORT-->

Co-authored-by: Navarone Feekery <[email protected]>
kibanamachine added a commit that referenced this pull request Dec 6, 2024
…03216)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[Search][a11y] Fix table row screen reader error
(#203136)](#203136)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Navarone
Feekery","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-06T09:10:26Z","message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-major"],"title":"[Search][a11y]
Fix table row screen reader
error","number":203136,"url":"https://github.com/elastic/kibana/pull/203136","mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203136","number":203136,"mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}}]}]
BACKPORT-->

Co-authored-by: Navarone Feekery <[email protected]>
kibanamachine added a commit that referenced this pull request Dec 6, 2024
…03218)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[Search][a11y] Fix table row screen reader error
(#203136)](#203136)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Navarone
Feekery","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-06T09:10:26Z","message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:prev-major"],"title":"[Search][a11y]
Fix table row screen reader
error","number":203136,"url":"https://github.com/elastic/kibana/pull/203136","mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203136","number":203136,"mergeCommit":{"message":"[Search][a11y]
Fix table row screen reader error (#203136)\n\n## Closes
https://github.com/elastic/kibana/issues/199113\r\n\r\nAllows errors in
the table row to be read by screen
readers.","sha":"1a9893bd6f7fd3fb94b08459e05921a00d86b8c5"}}]}]
BACKPORT-->

Co-authored-by: Navarone Feekery <[email protected]>
markov00 pushed a commit to markov00/kibana that referenced this pull request Dec 7, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Dec 9, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 9, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/kibana that referenced this pull request Dec 10, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
mykolaharmash pushed a commit to mykolaharmash/kibana that referenced this pull request Dec 11, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
##  Closes elastic#199113

Allows errors in the table row to be read by screen readers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-major Backport to (8.x, 8.17, 8.16) the previous major branch and other branches in development release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.15.6 v8.16.2 v8.17.0 v8.18.0 v9.0.0
Projects
None yet
4 participants