Skip to content

Fix broken Header layout on small screens#3798

Merged
Dino-ONS merged 2 commits intomainfrom
feature/812/header-menu-search-alignment
Mar 18, 2026
Merged

Fix broken Header layout on small screens#3798
Dino-ONS merged 2 commits intomainfrom
feature/812/header-menu-search-alignment

Conversation

@Dino-ONS
Copy link
Copy Markdown
Contributor

@Dino-ONS Dino-ONS commented Mar 13, 2026

What is the context of this PR?

The ONS header navigation controls, specifically Menu and Search, become visually misaligned, appearing out of place ONSDESYS-812.

An update to the menu-links class within header.scss was applied to allow for better alignment.

How to review this PR

Go to /components/header/example-header-basic-with-search-and-language and use the toggle device toolbar to reduce the dimensions to iPhone SE. The Menu and Search buttons should be aligned vertically as shown in the screenshots below.

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@Dino-ONS Dino-ONS self-assigned this Mar 13, 2026
@Dino-ONS Dino-ONS requested a review from a team as a code owner March 13, 2026 15:48
@Dino-ONS Dino-ONS added the Bug Something isn't working label Mar 13, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
design-system Ready Ready Preview, Comment Mar 18, 2026 1:56pm

@SriHV
Copy link
Copy Markdown
Contributor

SriHV commented Mar 16, 2026

Screenshot 2026-03-16 at 15 54 24

When I set the viewport width to 320px, I see a minor overlap. Should we address this?

@Dino-ONS
Copy link
Copy Markdown
Contributor Author

Thanks Sri. After todays meeting, they were happy to go ahead with the 375px width fix until we move the language link location for mobile in a new task.

@sveltifier sveltifier changed the title Header with menu and search button responsiveness issue Fix broken Header layout on small screens Mar 18, 2026
@Dino-ONS Dino-ONS merged commit e7c4675 into main Mar 18, 2026
12 checks passed
@Dino-ONS Dino-ONS deleted the feature/812/header-menu-search-alignment branch March 18, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants