Crown-Commercial-Service / digitalmarketplace-admin-frontend

Frontend administration application for the digital marketplace
MIT License
5 stars 15 forks source link

Links and buttons in summary tables require extra hidden text #557

Open lfdebrux opened 4 years ago

lfdebrux commented 4 years ago

When we have a summary table with a number of buttons/links we tend to repeat the link and expect a user to be able to associate the button with the text of the row.

However, for a user using a screenreader, these button/link names are ambiguous and unclear.

For screenreaders to be able to give users context on what a link or button actually does we need to include hidden text to expand on what the button does.

Checklist

lfdebrux commented 4 years ago

I've added a checklist for all the places where we use the summary table; I've ticked off the ones which don't seem to have a button or a link.

katstevens commented 4 years ago

We have an existing pattern for this that probably got missed when building the admin tools, a hidden_text attribute on summary.edit_link where extra context can be added for screen readers: https://github.com/alphagov/digitalmarketplace-briefs-frontend/blob/master/app/templates/buyers/section_summary.html#L75