Remove title attribute from summary list

[andysellick]

What

Remove the title attributes from the links in the summary list component. These are automatically generated based on the link text and the thing they refer to, or passed to the component directly (a change I recently added).

Why

We don't usually use the title attribute for links on GOV.UK, instead relying on visually hidden text within the link text. What's strange is that this component already does this, so the title attribute seems an unnecessary addition, particularly when the original component in the Design System doesn't have title attributes on links - https://design-system.service.gov.uk/components/summary-list/.

There's also evidence that screen readers treat it in various ways. Specifically https://www.deque.com/blog/text-links-practices-screen-readers/ highlights this, and this article https://www.powermapper.com/tests/screen-readers/labelling/a-title/ shows that it can cause problems in 17 screen reader / browser combinations.

Visual Changes

None.

[alex-ju]

I did a bit of investigation in an attempt to shed a bit of light on how we ended up with this. Content Publisher (then later Collection Publisher), by design, needed a summary component. At the time this was only available as a pattern in the prototype kit. The component was ported from the prototype kit into Content Publisher, where title referred to the section title – it seems that over time this attribute morphed into being used as an HTML title attribute – hence the redundant element.

Happy for this to be removed as we have the visually hidden element.

[danacotoran]

Just one comment, I think it looks good otherwise.

[danacotoran]

Thanks for updating the doc with my suggestion 👍