carpentries / varnish

Template for pkgdown site
https://carpentries.github.io/varnish/
Other
7 stars 29 forks source link

Tables should use `display: table` #73

Closed zkamvar closed 1 year ago

zkamvar commented 1 year ago

Right now, our table elements use display: block, which goes against accessibility recommendations for tables to not use block display (emphasis mine):

Changing the display value of a <table> element to block, grid, or flex will alter its representation in the accessibility tree. This will cause the table to no longer be announced properly by screen reading technology.

Thus line 156 here should be changed to display: table:

https://github.com/carpentries/varnish/blob/4385602d3e1f49a8918e15def97180dc27252b77/source/stylesheets/styles.css.scss#L155-L159