No Code Attached Yet a11y
avatar brianteeman
brianteeman
23 Feb 2020

There are a million and one reasons why links should not open in a new window link targets

TL;DR: Regardless of what accessibility conformance level you target, do not arbitrarily open links in a new window or tab. If you are required to do so anyway, inform users in text.

We are currently visually indicating this but we are not indicating it to screen readers.

You might think that screen readers were intelligent enough to recognise a target=_blank and say something to the user but they don't powermapper results

Elsewhere I have changed the code so that link has "title=open LINKNAME in new window"

For com_csp and plg_system_httpheaders it's not that easy.

The language string contains the link
PLG_SYSTEM_HTTPHEADERS_HSTS="<a href='https://hstspreload.org' target='_blank' rel='noopener noreferrer'>HTTP Strict Transport Security (HSTS)</a>"

While I could update the string with the title the problem is that we have told translators NOT to translate the string.

Anyone have any clever ideas how to resolve this?

avatar brianteeman brianteeman - open - 23 Feb 2020
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Feb 2020
avatar brianteeman brianteeman - change - 23 Feb 2020
The description was changed
avatar brianteeman brianteeman - edited - 23 Feb 2020
avatar jwaisner jwaisner - change - 23 Feb 2020
Status New Confirmed
Build staging 4.0-dev
avatar zero-24
zero-24 - comment - 24 Feb 2020

Isn't there a CSS class to add that external link indication? I don't remember wich it was.

If no other option works we can change the message to the translators ;-)

avatar brianteeman
brianteeman - comment - 24 Feb 2020

It's css so it's just a visual indication

avatar zero-24
zero-24 - comment - 24 Feb 2020

Well i guess than we have to update the links / strings and update the translator message.

avatar brianteeman
brianteeman - comment - 24 Feb 2020

I am thinking of doing it a different way

Keep the labels as text only labels
ADD links as the descriptions

avatar zero-24
zero-24 - comment - 24 Feb 2020

Go for it when it is better for accessibility, top. I think this are a few settings where deeper knowledge / descriptions makes sense.

avatar brianteeman
brianteeman - comment - 24 Feb 2020

will put it on the todo list

avatar brianteeman
brianteeman - comment - 7 Apr 2020

Maybe the Accessibility team will do it. I don't have time

avatar alikon alikon - change - 7 Apr 2020
Labels Added: ?
avatar alikon alikon - labeled - 7 Apr 2020
avatar brianteeman brianteeman - change - 1 Feb 2022
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2022-02-01 23:09:08
Closed_By brianteeman
Labels Added: No Code Attached Yet a11y
Removed: ? ?
avatar brianteeman brianteeman - close - 1 Feb 2022

Add a Comment

Login with GitHub to post a comment