Skip to content

[A11y][APM] Improve aria-label for EuiCodeBlock on onboarding page#217292

Merged
rmyz merged 3 commits intoelastic:mainfrom
rmyz:195030-a11y-apm-improve-onboarding-aria-label
Apr 9, 2025
Merged

[A11y][APM] Improve aria-label for EuiCodeBlock on onboarding page#217292
rmyz merged 3 commits intoelastic:mainfrom
rmyz:195030-a11y-apm-improve-onboarding-aria-label

Conversation

@rmyz
Copy link
Contributor

@rmyz rmyz commented Apr 7, 2025

Summary

Fixes #195030

This PR adds aria-label with language context for EuiCodeBlock in the APM onboarding page.

@rmyz rmyz added release_note:fix apm Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. backport:version Backport to applied version labels a11y Accessibility issue v8.19.0 labels Apr 7, 2025
@rmyz rmyz self-assigned this Apr 7, 2025
@rmyz rmyz changed the title [A11y][APM] Improve aria-label for EuiCodeBlock on onboarding page [A11y][APM] Improve aria-label for EuiCodeBlock on onboarding page Apr 7, 2025
@rmyz rmyz marked this pull request as ready for review April 7, 2025 10:31
@rmyz rmyz requested a review from a team April 7, 2025 10:31
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

Copy link
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

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

The code change LGTM

@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
apm 2.6MB 2.6MB +1.9KB

History

cc @rmyz

@rmyz rmyz merged commit a49dc03 into elastic:main Apr 9, 2025
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.18, 8.x, 9.0

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

@rmyz rmyz deleted the 195030-a11y-apm-improve-onboarding-aria-label branch April 9, 2025 13:57
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.18 Backport failed because of merge conflicts
8.x Backport failed because of merge conflicts
9.0 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 217292

Questions ?

Please refer to the Backport tool documentation

rmyz added a commit to rmyz/kibana that referenced this pull request Apr 9, 2025
elastic#217292)

## Summary

Fixes elastic#195030

This PR adds `aria-label` with language context for `EuiCodeBlock` in
the APM onboarding page.

(cherry picked from commit a49dc03)

# Conflicts:
#	x-pack/solutions/observability/plugins/apm/public/components/app/onboarding/instructions_set.tsx
@rmyz
Copy link
Contributor Author

rmyz commented Apr 9, 2025

💚 All backports created successfully

Status Branch Result
9.0
8.x
8.18

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

Questions ?

Please refer to the Backport tool documentation

rmyz added a commit to rmyz/kibana that referenced this pull request Apr 9, 2025
elastic#217292)

## Summary

Fixes elastic#195030

This PR adds `aria-label` with language context for `EuiCodeBlock` in
the APM onboarding page.

(cherry picked from commit a49dc03)

# Conflicts:
#	x-pack/solutions/observability/plugins/apm/public/components/app/onboarding/instructions_set.tsx
rmyz added a commit to rmyz/kibana that referenced this pull request Apr 9, 2025
elastic#217292)

## Summary

Fixes elastic#195030

This PR adds `aria-label` with language context for `EuiCodeBlock` in
the APM onboarding page.

(cherry picked from commit a49dc03)

# Conflicts:
#	x-pack/solutions/observability/plugins/apm/public/components/app/onboarding/instructions_set.tsx
rmyz added a commit that referenced this pull request Apr 9, 2025
…ng page (#217292) (#217689)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[A11y][APM] Improve `aria-label` for `EuiCodeBlock` on onboarding
page (#217292)](#217292)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-04-09T13:55:12Z","message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","apm","Team:obs-ux-infra_services","backport:version","a11y","v8.18.0","v9.1.0","v8.19.0","v9.0.1"],"title":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding
page","number":217292,"url":"https://github.com/elastic/kibana/pull/217292","mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","8.x","9.0"],"targetPullRequestStates":[{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/217292","number":217292,"mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
rmyz added a commit that referenced this pull request Apr 9, 2025
…ing page (#217292) (#217691)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[A11y][APM] Improve `aria-label` for `EuiCodeBlock` on onboarding
page (#217292)](#217292)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-04-09T13:55:12Z","message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","apm","Team:obs-ux-infra_services","backport:version","a11y","v8.18.0","v9.1.0","v8.19.0","v9.0.1"],"title":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding
page","number":217292,"url":"https://github.com/elastic/kibana/pull/217292","mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","8.x","9.0"],"targetPullRequestStates":[{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/217292","number":217292,"mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
rmyz added a commit that referenced this pull request Apr 10, 2025
…ng page (#217292) (#217690)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[A11y][APM] Improve `aria-label` for `EuiCodeBlock` on onboarding
page (#217292)](#217292)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-04-09T13:55:12Z","message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","apm","Team:obs-ux-infra_services","backport:version","a11y","v8.18.0","v9.1.0","v8.19.0","v9.0.1"],"title":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding
page","number":217292,"url":"https://github.com/elastic/kibana/pull/217292","mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","8.x","9.0"],"targetPullRequestStates":[{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/217292","number":217292,"mergeCommit":{"message":"[A11y][APM]
Improve `aria-label` for `EuiCodeBlock` on onboarding page
(#217292)\n\n## Summary\n\nFixes #195030\n\nThis PR adds `aria-label`
with language context for `EuiCodeBlock` in\nthe APM onboarding
page.","sha":"a49dc033306931b83facd68b062b30339ec895ed"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility issue apm backport:version Backport to applied version labels release_note:fix Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v8.18.0 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Home > Applications > Services > Add Data -- Whichever tab is selected, 'Copy' buttons do not convey relationships to their specific functions

5 participants