? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
30 Jan 2021

Due to some "bugs" in some screenreaders and browsers and how they handle svg it is recommended to set them to focusable=false and ensure they have the correct role.

Because of the way we handle svg logos in the template this has to be done in the js and not the svg file itself.

The only way to really test after applying the PR and npm i is to view the source. If successable the svg will have the role and focusable attributes as below

image

avatar brianteeman brianteeman - open - 30 Jan 2021
avatar brianteeman brianteeman - change - 30 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Jan 2021
Category JavaScript Repository NPM Change
avatar ceford
ceford - comment - 31 Jan 2021

This is only working with Debug System = Yes in Global Configuration. Is that expected at this stage?


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

avatar brianteeman
brianteeman - comment - 31 Jan 2021

No I cannot replicate that

avatar ceford
ceford - comment - 31 Jan 2021

I ran npm ci and then checked template.js and template.min.js
template.js contains what I expect and works. template.min.js contains f.setAttribute("role","presentation"),f.setAttribute("focusable","false") and does not work in Firefox or Chrome on Mac. Both files have the same date/time consistent with the npm compilation. Also I have template.es6 versions. Does all that mean there is something amiss with my npm compilation? I don't see any javascript errors.

avatar ceford ceford - test_item - 31 Jan 2021 - Tested successfully
avatar ceford
ceford - comment - 31 Jan 2021

I have tested this item successfully on 5bc9449

I traced the problem I reported to the presence of template.min.js.gz dated to a few days ago. Deleting that file fixed the problem.


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

avatar gostn gostn - test_item - 1 Feb 2021 - Tested successfully
avatar gostn
gostn - comment - 1 Feb 2021

I have tested this item successfully on 5bc9449


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

avatar infograf768
infograf768 - comment - 1 Feb 2021

Question: do all svg need to be modified this way?
Example: the webauthn.svg

avatar brianteeman
brianteeman - comment - 1 Feb 2021

Ask the accessibility team

avatar Quy Quy - change - 3 Feb 2021
Status Pending Ready to Commit
avatar Quy
Quy - comment - 3 Feb 2021

RTC


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

avatar chmst chmst - change - 4 Feb 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-02-04 15:02:24
Closed_By chmst
Labels Added: ? ?
avatar chmst chmst - close - 4 Feb 2021
avatar chmst chmst - merge - 4 Feb 2021
avatar chmst
chmst - comment - 4 Feb 2021

Thanks!

avatar brianteeman
brianteeman - comment - 4 Feb 2021

thanks

Add a Comment

Login with GitHub to post a comment