Plugin names should be included within links in Install Plugins

[GrahamArmfield]

In the new Install Plugins screen there are links that carry the text 'Update Now' or 'Install Now'. There are also links that carry text 'More Details'.

None of these links contain the name of the plugins. So screen reader users using the list links functionality are getting no context to let them know which plugin is being referenced by the links.

The plugin name needs to be included in the links.

There is an aria-labelledby attribute in the links that refers to a non-existant element. Obviously someone has attempted to get some context for the links from this.

Unfortunately, the behaviour of aria-labelledby and aria-label when used on links is to override the link text. So if the current approach did work, the two links for each plugin would actually be identical for screen reader users, when obviously they are not.

In my view, and to cater for internationalisation I suggest this approach:

<a href="xxx" aria-label="Install plugin name now">Install now</a>

and the same approach for the More details link.

Sighted users won't see any difference, but screen reader users will hear the full link with context.

[nvwd]

changed aria-labelledby to aria-label and added more descriptive text to be used by screen readers

[nvwd]

Text added to aria-label attribute

• Install {plugin title} now
• Update {plugin title} now
• More details about {plugin title}

[ocean90]

29018.patch​ doesn't work for i18n. Needs sprintf() if we want to go this way.

There is an aria-labelledby attribute in the links that refers to a non-existant element.

The attribute was added in [29201], see #28853. Maybe we should re-open the ticket?

[nvwd]

I will make the changes to the patch using sprintf().
This does seem to be part of #28853. I'll wait to attach the updated patch until a decision is reached about re-opening the original ticket.

[ocean90]

Duplicate of #28853.