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

Bug - Table - High Visibility Accessibility Issue #9969

Closed
Arathy-s opened this issue Jan 15, 2024 · 0 comments · Fixed by #10152
Closed

Bug - Table - High Visibility Accessibility Issue #9969

Arathy-s opened this issue Jan 15, 2024 · 0 comments · Fixed by #10152
Assignees

Comments

@Arathy-s
Copy link

Describe the problem
An unexpected table header element transformation when implementing the 'select all' functionality for checkboxes. Upon attaching a 'select' event handler to the <Th> element within the checkbox column's header, the <Th> element is converted to a <Td> element in the DOM.
Compromising the table structure creates an accessibility issue as there is no table header(<Th>) for a column.

How do you reproduce the problem?
Please try to inspect the structure of this table in the DOM.
https://www.patternfly.org/components/table/#selectable-with-checkbox

Expected behaviour
The table structure should not be altered on rendering in the DOM.

Is this issue blocking you?
https://issues.redhat.com/browse/PCM-12748

Screenshots
Screenshot 2024-01-12 at 3 16 28 PM

What is your environment?

  • OS: Mac OS Sonoma
  • Browser: Chrome
  • Version: 120.0.6099.216

What is your product and what release date are you targeting?
Customer Portal

Any other information?
As part of the Customer Portal accessibility initiative, we are fixing all the accessibility issues throughout the Customer Portal. So this issue is a high-priority one as it is affecting the accessibility score on all pages which use this table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants