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:[a11y sev2] OnRowClick is not triggered on keyboard enter in DataGrid #2540

Closed
adamint opened this issue Aug 16, 2024 · 0 comments · Fixed by #2577
Closed

fix:[a11y sev2] OnRowClick is not triggered on keyboard enter in DataGrid #2540

adamint opened this issue Aug 16, 2024 · 0 comments · Fixed by #2577
Labels
status:needs-investigation Needs additional investigation

Comments

@adamint
Copy link
Member

adamint commented Aug 16, 2024

🐛 Bug Report

It is required to provide a keyboard-accessible way to perform actions. In addition to clicking a row, OnRowClick must be called on keyboard enter on any cell in the row. Additionally, there must be some aria information on generated cells that signals to a screen reader that the cell is acting as a button.

💻 Repro or Code Sample

Filed as a bug report since it's an accessibility blocker, but really a feature request so no repro.

🤔 Expected Behavior

n/a

😯 Current Behavior

n/a

💁 Possible Solution

n/a

🔦 Context

WCAG 2.1 requirement

🌍 Your Environment

  • OS & Device: [e.g. MacOS, iOS, Windows, Linux] on [iPhone 7, PC]
  • Browser [e.g. Microsoft Edge, Google Chrome, Apple Safari, Mozilla FireFox]
  • .NET and Fluent UI Blazor library Version [e.g. 8.0.2 and 4.4.1]
@microsoft-github-policy-service microsoft-github-policy-service bot added the triage New issue. Needs to be looked at label Aug 16, 2024
@vnbaaij vnbaaij added status:needs-investigation Needs additional investigation and removed triage New issue. Needs to be looked at labels Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:needs-investigation Needs additional investigation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants