search

nhsuk / nhsuk-frontend

NHS.UK frontend contains the code you need to start building user interfaces for NHS websites and services.
https://nhsuk.github.io/nhsuk-frontend/
MIT License
606 stars 106 forks source link

Remove .nhsuk-u-top-and-bottom utility class #979

Closed frankieroberto closed 1 week ago

frankieroberto commented 1 week ago

I spotted that the .nhsuk-u-top-and-bottom utility class isn't actually included within the build as it was removed from core/all.scss in 2018 (https://github.com/nhsuk/nhsuk-frontend/commit/df303c1f342f0365707234c9e1b6f4c7feab6b51).

From the looks of it, the top-and-bottom() mixin was likely only intended for use within other components, and I can’t see any evidence of it having been used as a standalone utility class (see GitHub search.

So I think this is safe to remove?

Checklist

sarawilcox commented 1 week ago

Hi @roshaanbajwa, were you going to check this one?