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: [Obs Synthetics > Monitor test run][KEYBOARD]: Date (check) dropdown needs a semantic label for the current check #187353

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

alexwizp
Copy link
Contributor

@alexwizp alexwizp commented Jul 2, 2024

Closes: https://github.com/elastic/observability-dev/issues/3689

Description

The synthetics monitors include thumbnail screenshots that open a larger preview window. These thumbnails must take keyboard focus, manage the Enter and Space keypresses to open the modal, and return focus to the originating thumbnail when the modal is closed. Screenshots attached below.

Steps to recreate

  1. Open the Synthetics view
  2. Create a monitor if none exist
  3. Click on that monitor and navigate to the full monitor detail
  4. Click the "View test run link" in the Last test run module (about 2/3 of the way down the page)
  5. Open the "Date" dropdown in the top right
  6. Either listen to the dropdown nodes with a screen reader or evaluate with Dev Tools Accessibility Tree. This will show you a standard, non-semantic div for the current check.

What was changed?:

  1. aria-label attribute was added to address that issue
  2. similar changes were applied to 3 places:
    • x-pack/plugins/observability_solution/synthetics/public/apps/synthetics/components/step_details_page/step_page_nav.tsx
    • x-pack/plugins/observability_solution/uptime/public/legacy_uptime/components/monitor/synthetics/step_detail/step_page_nav.tsx
    • x-pack/plugins/observability_solution/uptime/public/legacy_uptime/pages/synthetics/checks_navigation.tsx

Screen

image

@alexwizp
Copy link
Contributor Author

alexwizp commented Jul 2, 2024

/ci

@alexwizp alexwizp added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting v8.15.0 Project:Accessibility labels Jul 2, 2024
@alexwizp alexwizp marked this pull request as ready for review July 2, 2024 13:22
@alexwizp alexwizp requested a review from a team as a code owner July 2, 2024 13:22
@elasticmachine
Copy link
Contributor

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

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

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
synthetics 1.0MB 1.0MB +168.0B
uptime 467.2KB 467.6KB +357.0B
total +525.0B

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

Copy link
Contributor

@MiriamAparicio MiriamAparicio left a comment

Choose a reason for hiding this comment

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

LGTM

@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jul 3, 2024
@elasticmachine
Copy link
Contributor

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

@alexwizp alexwizp merged commit ce4f469 into elastic:main Jul 3, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:review backport:skip This commit does not require backporting ci:project-deploy-observability Create an Observability project Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants