search

wenzhixin / bootstrap-table

An extended table to integration with some of the most widely used CSS frameworks. (Supports Bootstrap, Semantic UI, Bulma, Material Design, Foundation, Vue.js)
https://bootstrap-table.com/
MIT License
11.73k stars 4.44k forks source link

empty table should have role="presentation" per WCAG accessibility guidelines #7415

Closed tailcat closed 1 month ago

tailcat commented 1 month ago

Description

Line 115 of src/bootstrap-table.js has:

The table should have role="presentation" per WCAG guidelines.

https://webaim.org/standards/wcag/checklist#sc1.3.1

wenzhixin commented 1 month ago

What is the purpose of this attribute? Could you give an example?

tailcat commented 1 month ago

<table role="presentation">

this tells a screen reader to not think of this as a table of things but a table just used for its positioning properties.

UtechtDustin commented 1 month ago

Makes perfect sense, no question needed.
Opened a PR for that :)