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

Accessibility: Improve keyboard accessibility in AnnoListPanel #58971

Merged
merged 1 commit into from Nov 21, 2022

Conversation

ashharrison90
Copy link
Contributor

What is this feature?

  • uses Button to remove an applied user filter instead of a <span> with an onClick
  • removes FocusScope which prevents shift-tabbing in the AnnoListPanel

Why do we need this feature?

  • better keyboard experience

Who is this feature for?

  • everyone! 🙌

Which issue(s) does this PR fix?:

Fixes #58969

Special notes for your reviewer:

@ashharrison90 ashharrison90 added this to the 9.3.0 milestone Nov 18, 2022
@ashharrison90 ashharrison90 requested a review from a team November 18, 2022 14:46
@ashharrison90 ashharrison90 self-assigned this Nov 18, 2022
@ashharrison90 ashharrison90 requested review from joshhunt and JoaoSilvaGrafana and removed request for a team November 18, 2022 14:46
@grafanabot
Copy link
Contributor

Copy link
Contributor

@JoaoSilvaGrafana JoaoSilvaGrafana left a comment

Choose a reason for hiding this comment

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

That FocusScope bug is quite funny 😂

Good job! 👏

@ashharrison90 ashharrison90 merged commit 598935c into main Nov 21, 2022
@ashharrison90 ashharrison90 deleted the ash/58969 branch November 21, 2022 09:25
grafanabot pushed a commit that referenced this pull request Nov 21, 2022
)

fix keyboard accessibility in AnnoListPanel

(cherry picked from commit 598935c)
ashharrison90 added a commit that referenced this pull request Nov 21, 2022
…nel` (#59015)

Accessibility: Improve keyboard accessibility in `AnnoListPanel` (#58971)

fix keyboard accessibility in AnnoListPanel

(cherry picked from commit 598935c)

Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
GuYounes pushed a commit to paul-wurth/BIXpert that referenced this pull request Feb 8, 2023
…nel` (grafana#59015)

Accessibility: Improve keyboard accessibility in `AnnoListPanel` (grafana#58971)

fix keyboard accessibility in AnnoListPanel

(cherry picked from commit 598935c)

Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A11y: Fix keyboard accessibility in public/app/plugins/panel/annolist/AnnoListPanel.tsx
3 participants