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.

Sign up for GitHub

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

Jump to bottom

[Bug][a11y]: Virtualized Table should include aria-rowcount attribute #3393

Closed
1 task done
majornista opened this issue Jun 29, 2023 · 0 comments · Fixed by #4126 or #4156
Closed
1 task done

[Bug][a11y]: Virtualized Table should include aria-rowcount attribute #3393

majornista opened this issue Jun 29, 2023 · 0 comments · Fixed by #4126 or #4156
Labels
a11y Issues related to accessibility bug Something isn't working Component: Table good first issue Good for newcomers help wanted Extra attention is needed VoiceOver

Comments

@majornista
Copy link
Contributor

Code of conduct

  • I agree to follow this project's code of conduct.

Impacted component(s)

Table

Expected behavior

When a Table is virtualized it should include an aria-rowcount attribute to communicate the number of total rows in the table, otherwise screen readers will announce only the number of rendered rows.

Actual behavior

Virtualized table lacks aria-rowcount attribute, so screen readers will announce only the number of rendered rows rather than the total rows in the table.

Screenshots

No response

What browsers are you seeing the problem in?

No response

How can we reproduce this issue?

  1. Go to '...'
  2. Click on '....'
  3. Scroll to '....'
  4. Check console
  5. See error

Sample code that illustrates the problem

No response

Logs taken while reproducing problem

No response
@majornista majornista added bug Something isn't working triage An issue needing triage Component: Table a11y Issues related to accessibility labels Jun 29, 2023
@najikahalsema najikahalsema added VoiceOver and removed triage An issue needing triage labels Oct 26, 2023
@Westbrook Westbrook added help wanted Extra attention is needed good first issue Good for newcomers labels Mar 4, 2024
@blaabaer blaabaer mentioned this issue Mar 5, 2024
Merged
13 tasks
@blaabaer blaabaer linked a pull request Mar 5, 2024 that will close this issue
fix(table): Add aria-rowcount to virtualized tables #4126
Merged
13 tasks
@blaabaer blaabaer mentioned this issue Mar 7, 2024
Merged
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
a11y Issues related to accessibility bug Something isn't working Component: Table good first issue Good for newcomers help wanted Extra attention is needed VoiceOver
Projects
None yet
Milestone
No milestone
3 participants
@majornista @Westbrook @najikahalsema