User tests: Successful: Unsuccessful:
JAT has found several a11y issues in the SystemInformation View:
Remove the < form > tag, as this is not a form.
Remove empty Table Footers
Replace the < fieldset > Tag by < div >
Replace the < legend > by Table < caption >
There should be no visible difference in the View System Information before and after patch.
Test of accessibility with screenreader is required.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_admin |
Category | Administration com_admin | ⇒ | Accessibility Administration com_admin |
Thank ou @brianteeman. What do you think about adding the "caption-side: top" into one of the general scss files? If there are more tables, not only in com_admin, they all could have captions.
Sure. That would work as well
Correction to my last comment -
although confusingly named
atum\scss\vendor\bootstrap
is not their code it is our overrides of their code. So I would put it in the _table.scss file
Labels |
Added:
J4 Issue
|
Category | Administration com_admin Accessibility | ⇒ | Administration com_admin Templates (admin) Accessibility |
Labels |
Added:
?
|
To say the thruth: I think it does not need a caption or headline at all in this screen. The caption repeats the name of the tab and gives no information.
To say the thruth: I think it does not need a caption or headline at all in this screen. The caption repeats the name of the tab and gives no information.
Just checked other tabbed interfaces in joomla and they dont have titles either so you are correct
If those strings are only.used here then they can be removed from the language files
This are the Tab names. They were repeated in the Headline / caption.
ok
I have tested this item
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-08-19 14:47:21 |
Closed_By | ⇒ | franz-wohlkoenig |
Closed_Date | 2018-08-19 14:47:21 | ⇒ | 2018-08-19 14:47:22 |
Closed_By | franz-wohlkoenig | ⇒ | joomla-cms-bot |
closed as requested at #21724
Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/21628
unfortunately thats not correct - without some markup for the caption element the caption is displayed in muted text at the bottom of the screen.
I would create a new sscss file here
administrator\templates\atum\scss\pages\_com_admin.scss
and add the markup there.This will add the caption to the top - then you just need to address the font-size and wieght
The rest looks good