ministryofjustice / find-moj-data

Find MOJ data service • This repository is defined and managed in Terraform
MIT License
5 stars 0 forks source link

Fix colons and tabulation in list results #194

Closed jemnery closed 4 months ago

jemnery commented 7 months ago

Change to match the patterns in figma. Fix on the seach results and details page.

The content of terms in description lists have inconsistent use of colons.

The examples in the design system don't use colons. I assume because the <dl> itself creates logical separation and we don't need colons.

Can we create a Selenium test for this? 🤔

MatMoore commented 7 months ago

@jemnery I think this inconsistency has come about because we've diverged from the design.

The figma doesn't use the summary list component, and instead just has one item per line with a colon. I think we just implemented this pattern of our own accord early on and never updated it, and the one in the figma is how it should be.

https://www.figma.com/file/zTOVhHRnWfxRJqH6Gr8cIc/Catalogue-sketches?type=design&node-id=203-2121&mode=design&t=YocOJ80P7hF7KanZ-0

Previously I corrected this on the details page by using a govuk list instead of govuk summary list. The result looks like this:

Image

I reckon we should do the same thing on the search results page. Not using the summary list also means we can use a horizontal line break to separate the actual results, as in this DFE example:

Image

What do you think @alex-vonfeldmann ?

jemnery commented 7 months ago

I like the idea of using a <hr /> to separate the results

YvanMOJdigital commented 7 months ago

wait for ur playback

YvanMOJdigital commented 6 months ago

Remove tags/retention period/first created from results view.

Need more research to better understand what people want from results page.