wellcomecollection / wellcomecollection.org

🪟 Wellcome Collection's website and services that support it
https://wellcomecollection.org
MIT License
37 stars 5 forks source link

Create a text highlight element for the Visual Story page body contents #10885

Closed kasiabbb closed 2 weeks ago

kasiabbb commented 1 month ago

User Story

As a neurodivergent user accessing a visual story I want to see what other visual stories there are available So that I can be confident planning a visit to exhibitions at Wellcome Collections

What?

Add the yellow highlight element to every Visual Story link inside another Visual Story

Image

Why?

Currently links buried inside the guide, not very visible.

Background

Done when

rcantin-w commented 1 month ago

We discussed it with Lauren and Kasia and we're happy doing this, but we'll need to make it very clear what the slice is to be used for. Historically, InfoBlock has been used for an array of reason and we'd like to ensure it doesn't get used for more random things just because it stands out. So;

rcantin-w commented 3 weeks ago

Waiting on possible name change?

rcantin-w commented 3 weeks ago

Blocked by #10959

rcantin-w commented 2 weeks ago

https://wellcomecollection.org/exhibitions/ZZP8BxAAALeD00jo/visual-stories change made here. The InfoBlock is now available on visual stories and only offers title and content, no more buttons.

The three places we found that used the button have been replaced by a link.

Just noticed the styles are slightly different, with the list being indented on the mockup but not in our current InfoBlock styles. I'm not super sure why, it uses .body-text which overrides the padding on the left, so it does seem intentional. Is that something we'd like me to look into?

Image

rcantin-w commented 2 weeks ago

Changes communicated to Editorial in Slack.

Name of the component will not be changed as per this ticket's conclusion.

So closing this ticket - we can create a new one if we want to address the list styles; it's a different piece of work anyway.