wielebenwir / commonsbooking

CommonsBooking is an open source Wordpress plugin for sharing items with users. This is the NEW CommonsBooking (starting at version v2.0.0). Please install plugin via Wordpress plugin directory.
https://commonsbooking.org
GNU General Public License v2.0
47 stars 15 forks source link

View: accessiblity fix #1640

Open datengraben opened 4 weeks ago

datengraben commented 4 weeks ago

Ersetzt th mit td, weil der Inhalt immer leer zu sein scheint.

Das ist jetzt natürlich nur ein sehr kleiner Teil, von eigentlich recht vielen WAVE Webaim-Findings im Plugin. Der ist mir nur als recht einfach reparierbar aufgefallen.

Offene Fragen:

Hier der englische WAVE Webaim Bericht:

Empty table header

What It Means

A <th> (table header) contains no text.

Why It Matters

The <th> element helps associate table cells with the correct row/column headers. A <th> that contains no text may result in cells with missing or incorrect header information.

What To Do

If the table cell is a header, provide text within the cell that describes the column or row. If the cell is not a header or must remain empty (such as the top-left cell in a data table), make the cell a <td> rather than a <th>.

The Algorithm... in English

A <th> element does not contain any text (or contains only spaces) and no images with alternative text.

Standards and Guidelines

1.3.1 Info and Relationships (Level A)

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.23%. Comparing base (aedd156) to head (5c53426). Report is 22 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1640 +/- ## ============================================ - Coverage 50.24% 50.23% -0.01% - Complexity 2720 2722 +2 ============================================ Files 99 99 Lines 10862 10868 +6 ============================================ + Hits 5458 5460 +2 - Misses 5404 5408 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

hansmorb commented 2 weeks ago

Ersetzt th mit td, weil der Inhalt immer leer zu sein scheint.

Das ist jetzt natürlich nur ein sehr kleiner Teil, von eigentlich recht vielen WAVE Webaim-Findings im Plugin. Der ist mir nur als recht einfach reparierbar aufgefallen.

Offene Fragen:

* [ ]  Ist `$desc` immer leer bzw.  wird dort überhaupt Text sinnvoll dargestellt?

  * Falls ja, dann kann die Variable vielleicht auch raus.
  * Falls nein, muss es noch abgefragt werden.

Ja, aus der Dokumentation für den table shortcode:

Zusätzlich kann oberhalb der Tabelle eine kurze Beschreibung mit dem Attribut desc eingefügt werden. [cb_items_table desc=Lastenräder]

* [ ]  Gibt es Interesse alle Findings zu sammeln/taggen und systematisch abzuarbeiten?

  * Vielleicht mit einem issue label #accessibility oder kurz [#a11y](https://www.a11yproject.com/)

Können wir machen! Wir können uns auch nochmal austauschen weil solche Vorhaben auch förderungsfähig sind (zb. Aktion Mensch)

Hier der englische WAVE Webaim Bericht:

Empty table header

What It Means

A <th> (table header) contains no text.

Why It Matters

The <th> element helps associate table cells with the correct row/column headers. A <th> that contains no text may result in cells with missing or incorrect header information.

What To Do

If the table cell is a header, provide text within the cell that describes the column or row. If the cell is not a header or must remain empty (such as the top-left cell in a data table), make the cell a <td> rather than a <th>.

The Algorithm... in English

A <th> element does not contain any text (or contains only spaces) and no images with alternative text.

Standards and Guidelines

1.3.1 Info and Relationships (Level A)