search

Access4all / adg

Accessibility Developer Guide
http://www.accessibility-developer-guide.com
Other
187 stars 36 forks source link

ADG-308 Table spanning multiple rows and columns #430

Closed backflip closed 2 months ago

backflip commented 7 months ago

This PR finally implements Adrian's feedback in https://github.com/Access4all/adg/issues/308.

Demo

Notes

When testing, please make sure to review whether the Tricky screen reader navigation paragraph is still accurate.

netlify[bot] commented 7 months ago

Deploy Preview for accessibility-developer-guide ready!

Name Link
Latest commit 82fcfc068bf53416845639d63144b313def3530f
Latest deploy log https://app.netlify.com/sites/accessibility-developer-guide/deploys/66a20b3067ec640008989ac9
Deploy Preview https://deploy-preview-430--accessibility-developer-guide.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

christophdubach commented 6 months ago

@backflip: To me the source code additions look good and cover Adrian's feedback.

(But what I was wondering: Should the <th>s of the individual rows (those with the time specification) also have an id, and should this id also be listed in the headers attribute of the <td>s? Or does this only apply to header cells of columns?)

Important note: I cannot judge the correctness of the text section “Tricky screen reader navigation” or the compatibility table – my approval therefore only applies to the code ... 😉

NicSR commented 4 months ago

Sali Zäme

Nun ich habe jetzt mehrere Tests gemacht mit den Beispielen die wir anbieten und dem Beispiel von Adrian. In unseren Beispielen passiert gar nichts, und das Verhalten in Chrom/Firefox mit NVDA ist identisch. Es werden jeweils die Headers ausgelesen im thead-element drin unabhängig davon ob mit oder ohne ID.

Nun anders sieht es aus beim Beispiel von Adrian: https://adrianroselli.com/2022/01/accessible-cart-tables.html image da wird versucht, der Zelle "sub-total" die Header-Bezeichnung "Description" zu setzten. dies allerdings funktioniert nur in Firefox und NVDA nicht aber in Chrome und NVDA.

wollen wir dies als Beispiel mit im ADG aufnehmen um die Problematik stand heute zu zeigen?