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

Fix misaligned button and input on search component #3991

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matthillco
Copy link
Contributor

@matthillco matthillco commented Apr 23, 2024

What

Make the search input field and button the same height after applying type scale changes in design system v5.

Why

These two elements became misaligned when the input field font-size was increased as part of the design system typography improvements. We no longer need adjusted padding for the tablet+ view as the font size is now the same 19px at all screen sizes.

Visual Changes

  1. Original site with 16px fonts
  2. Updated typescale with 19px font and broken search field
  3. Updated typescale with 19px font and fixed search field (this PR)

image

These two elements became misaligned when the input field font-size
was increased as part of the design system typography improvements.

We no longer need adjusted padding for the tablet+ view as the font
size is now the same 19px at all screen sizes.
@matthillco matthillco added do not merge govuk-frontend Addresses misalignments with govuk-frontend in progress In progress issues are being worked on but may not be a high priority. labels Apr 23, 2024
@govuk-ci govuk-ci temporarily deployed to components-gem-pr-3991 April 23, 2024 17:55 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge govuk-frontend Addresses misalignments with govuk-frontend in progress In progress issues are being worked on but may not be a high priority.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants