Add 'visited link' indicator

[bryophyta]

What does this change?

• Store the ids of viewed items in state, and sync with localstorage
• Differentiate between items that have been viewed and that haven't:
  • Bolder text on unviewed items
  • Remove colour from supplier badge when the item has been viewed
  • Add visually hidden text indicating status, for screen reader users
• Other changes to balance out the new design:
  • Less blank space between the sections of the page (especially item view on desktop)
  • Adjust selection/hover styling for cards

nb. I looked at using the :visited pseudo-selector for this styling, to make use of the browser's built-in behaviour. However, this selector matches on the whole URL, including search params, which doesn't work for our purposes, because we don't want to treat /item/123?search=a as a separate page from /item/123?search=b. (The styling is meant ot indicate that you've viewed item 123.)

How to test

• Look at screenshots
• Deploy locally

How can we measure success?

Have we considered potential risks?

Images

	Before	After
Item selected
Item has been viewed
Page layout

Accessibility

• Tested with screen reader
• Navigable with keyboard
• Colour contrast passed
• The change doesn't use only colour to convey meaning

[bryophyta]

Having this as a warn was making the logs quite noisy when running tests. Given that we're not storing client-side logs in production, there doesn't seem much benefit to keeping this as a warn, as opposed to a log.

[github-actions]

Deploy build 816 of Editorial Tools::News Wires to CODE

All deployment options

• Deploy build 816 of Editorial Tools::News Wires to CODE
• Deploy parts of build 816 to CODE by previewing it first
• What's on CODE right now?

---

From guardian/actions-riff-raff.