? a11y bug PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
16 Mar 2023

Pull Request for Issue #.

Summary of Changes

To be accessible a table must have a header.
See for examle this checklist: https://dequeuniversity.com/checklists/web/tables

Joomla provides a param show_headings for frontend tables. This removes the table header.

This PR sets the table header visually-hidden so it is still present for assistive technologies.

I open some similar PRs to make testing easier.

Testing Instructions

You need at least one tag
Make a menu Item of Compact List of Tagged Items.
Set param table_headings in tab "list layouts" to "hide"
See the view and sourcecode ot the table in frontend

Actual result BEFORE applying this Pull Request

There is no table heading visible, also in source code.

Expected result AFTER applying this Pull Request

There is no table heading visible, but it is in sourcecode with class="visually-hidden"

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 16 Mar 2023
Category com_tags Front End
avatar chmst chmst - open - 16 Mar 2023
avatar chmst chmst - change - 16 Mar 2023
Status New Pending
avatar chmst chmst - change - 16 Mar 2023
Labels Added: a11y ?
avatar ChristineWk ChristineWk - test_item - 16 Mar 2023 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 16 Mar 2023

I have tested this item successfully on b7dd1d7


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40138.

avatar Quy Quy - test_item - 16 Mar 2023 - Tested successfully
avatar Quy
Quy - comment - 16 Mar 2023

I have tested this item successfully on b7dd1d7


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40138.

avatar Quy Quy - change - 16 Mar 2023
Status Pending Ready to Commit
avatar Quy
Quy - comment - 16 Mar 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40138.

avatar Quy Quy - change - 16 Mar 2023
Labels Added: ? PR-4.3-dev
avatar obuisard obuisard - change - 4 May 2023
Labels Added: bug
avatar obuisard obuisard - change - 4 May 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-04 20:45:57
Closed_By obuisard
avatar obuisard obuisard - close - 4 May 2023
avatar obuisard obuisard - merge - 4 May 2023
avatar obuisard
obuisard - comment - 4 May 2023

Thank you Christiane @chmst !

Add a Comment

Login with GitHub to post a comment