Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter panel UI tweaks #3466

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Filter panel UI tweaks #3466

merged 3 commits into from
Sep 23, 2024

Conversation

csutter
Copy link
Contributor

@csutter csutter commented Sep 20, 2024

  • Force checkbox/radio background
  • Fix hover underline
    • There was a stray space at the beginning of the button element which was causing the underline to cover more than just the text. This removes it and increases the chevron margin instead.
  • Give panel its own bottom border
    • The filter panel is always supposed to have a bottom border, but currently relies on the top border of the document list following it for its border. This gives the panel its own border so it's present even if there is no document list following it, and accordingly removes it from the top of the first document list item on the new all content finder view.

Before

image
image
image

After

image
image
image

@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3466 September 20, 2024 14:12 Inactive
GOV.UK Frontend radio and checkboxes are rendered with a transparent
background, which makes them look wrong on this component's grey
background. This is intentional in GOV.UK Frontend and unlikely to
change (see alphagov/govuk-frontend#1625).
There was a stray space at the beginning of the button element which was
causing the underline to cover more than just the text. This removes it
and increases the chevron margin instead.
@csutter csutter force-pushed the filter-panel-tweaks branch from 2a7f0ab to c419224 Compare September 20, 2024 14:13
@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3466 September 20, 2024 14:14 Inactive
The filter panel is always supposed to have a bottom border, but
currently relies on the top border of the document list following it for
its border.

This gives the panel its own border so it's present even if there is no
document list following it, and accordingly removes it from the top of
the first document list item on the new all content finder view.
@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3466 September 23, 2024 09:23 Inactive
@csutter csutter merged commit 90105e9 into main Sep 23, 2024
12 checks passed
@csutter csutter deleted the filter-panel-tweaks branch September 23, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants