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

fix: [Security:Cases] Sync alert status with case status toggle button under case settings on create case page is incorrectly announced #205604

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

alexwizp
Copy link
Contributor

@alexwizp alexwizp commented Jan 6, 2025

Closes: #205544

Description

When user tabs over sync alert status with case status toggle button under case settings on create case page, screenreader announces On, On switch without giving any context.

Preconditions

Security solution -> on cases page -> create case

Changes made:

  1. added context for EuiSwitch by passing aria-labelledby attribute

Screen

image

@alexwizp alexwizp added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting v9.0.0 backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Project:Accessibility and removed backport:skip This commit does not require backporting labels Jan 6, 2025
@alexwizp
Copy link
Contributor Author

alexwizp commented Jan 6, 2025

/ci

…n under case settings on create case page is incorrectly announced

Closes: elastic#205544
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
esUiShared 106.9KB 107.0KB +58.0B

History

@alexwizp alexwizp marked this pull request as ready for review January 7, 2025 14:23
@alexwizp alexwizp requested a review from a team as a code owner January 7, 2025 14:23
@elasticmachine
Copy link
Contributor

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

Copy link
Member

@sabarasaba sabarasaba left a comment

Choose a reason for hiding this comment

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

Thanks for patching this up @alexwizp, changes lgtm🥳

@alexwizp alexwizp merged commit efe44b2 into elastic:main Jan 8, 2025
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 8, 2025
…n under case settings on create case page is incorrectly announced (elastic#205604)

Closes: elastic#205544

## Description
When user tabs over sync alert status with case status toggle button
under case settings on create case page, screenreader announces On, On
switch without giving any context.

## Preconditions
Security solution -> on cases page -> create case

## Changes made:
1. added context for **EuiSwitch** by passing `aria-labelledby`
attribute

## Screen

![image](https://github.com/user-attachments/assets/94fb0413-d0c6-4a1c-9071-a6458e2cd379)

(cherry picked from commit efe44b2)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
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 Jan 8, 2025
… button under case settings on create case page is incorrectly announced (#205604) (#205938)

# Backport

This will backport the following commits from `main` to `8.x`:
- [fix: [Security:Cases] Sync alert status with case status toggle
button under case settings on create case page is incorrectly announced
(#205604)](#205604)

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

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

<!--BACKPORT [{"author":{"name":"Alexey
Antonov","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T16:57:41Z","message":"fix:
[Security:Cases] Sync alert status with case status toggle button under
case settings on create case page is incorrectly announced
(#205604)\n\nCloses: #205544\r\n\r\n## Description\r\nWhen user tabs
over sync alert status with case status toggle button\r\nunder case
settings on create case page, screenreader announces On, On\r\nswitch
without giving any context.\r\n\r\n## Preconditions\r\nSecurity solution
-> on cases page -> create case\r\n\r\n## Changes made: \r\n1. added
context for **EuiSwitch** by passing
`aria-labelledby`\r\nattribute\r\n\r\n##
Screen\r\n\r\n\r\n![image](https://github.com/user-attachments/assets/94fb0413-d0c6-4a1c-9071-a6458e2cd379)","sha":"efe44b2f2d97fd35f152fd93c5a2ffdcf57dbe2e","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","v9.0.0","backport:prev-minor"],"title":"fix:
[Security:Cases] Sync alert status with case status toggle button under
case settings on create case page is incorrectly
announced","number":205604,"url":"https://github.com/elastic/kibana/pull/205604","mergeCommit":{"message":"fix:
[Security:Cases] Sync alert status with case status toggle button under
case settings on create case page is incorrectly announced
(#205604)\n\nCloses: #205544\r\n\r\n## Description\r\nWhen user tabs
over sync alert status with case status toggle button\r\nunder case
settings on create case page, screenreader announces On, On\r\nswitch
without giving any context.\r\n\r\n## Preconditions\r\nSecurity solution
-> on cases page -> create case\r\n\r\n## Changes made: \r\n1. added
context for **EuiSwitch** by passing
`aria-labelledby`\r\nattribute\r\n\r\n##
Screen\r\n\r\n\r\n![image](https://github.com/user-attachments/assets/94fb0413-d0c6-4a1c-9071-a6458e2cd379)","sha":"efe44b2f2d97fd35f152fd93c5a2ffdcf57dbe2e"}},"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/205604","number":205604,"mergeCommit":{"message":"fix:
[Security:Cases] Sync alert status with case status toggle button under
case settings on create case page is incorrectly announced
(#205604)\n\nCloses: #205544\r\n\r\n## Description\r\nWhen user tabs
over sync alert status with case status toggle button\r\nunder case
settings on create case page, screenreader announces On, On\r\nswitch
without giving any context.\r\n\r\n## Preconditions\r\nSecurity solution
-> on cases page -> create case\r\n\r\n## Changes made: \r\n1. added
context for **EuiSwitch** by passing
`aria-labelledby`\r\nattribute\r\n\r\n##
Screen\r\n\r\n\r\n![image](https://github.com/user-attachments/assets/94fb0413-d0c6-4a1c-9071-a6458e2cd379)","sha":"efe44b2f2d97fd35f152fd93c5a2ffdcf57dbe2e"}}]}]
BACKPORT-->

Co-authored-by: Alexey Antonov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes v8.18.0 v9.0.0
Projects
None yet
4 participants