Skip to content

Commit

Permalink
pkp/pkp-lib#8880 Remove grid role and focus handling from Table compo…
Browse files Browse the repository at this point in the history
…nent

After reading a lot about the difference between tables and grids,
I've decided that our tables are not grids and should not implement
the grid role specification.

There is some ambiguity about when a table becomes interactive
enough to qualify as a grid. According to the W3C examples, a simple
table with an interactive element can be a grid. However, a lot of
the advice from accessibility consultants is to use a table unless
building a spreadsheet-like app.

For that reason, I decided it is probably best to keep the Table
component as a simple table role. Although we have some interactive
elements in our tables, they are still primarily presenting tabular
data. Following the principle of no aria is better than bad aria,
it is probably least harmful to avoid the grid specification for now.
  • Loading branch information
NateWr authored and Vitaliy-1 committed Aug 10, 2023
1 parent 7d8ac46 commit 2408892
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 223 deletions.
224 changes: 2 additions & 222 deletions src/components/Table/Table.vue
1 change: 0 additions & 1 deletion src/components/Table/TableCell.vue

0 comments on commit 2408892

Please sign in to comment.