Skip to content

fix: align mobile header search box - #17

Merged
Vardhan-IMMIDI merged 1 commit into
mainfrom
fix/mobile-header-search-alignment
Aug 30, 2026
Merged

Vardhan-IMMIDI merged 1 commit into
mainfrom
fix/mobile-header-search-alignment

Conversation

@varma1221

@varma1221 varma1221 commented Aug 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Closes #16 by correcting the mobile header layout so the search box no longer sits flush against the top edge of the viewport.

Changes

  • Added explicit two-row grid sizing for the mobile header.
  • Added responsive top and bottom padding to the mobile header container.
  • Explicitly placed the search area in the first grid row.
  • Vertically centered the search box within the mobile header row.
  • Preserved the existing mobile navigation row and desktop header layout.

Verification

  • npm run lint
  • npm run build
  • git diff --check
  • Verified on a narrow mobile viewport such as 375px.
  • Verified on a wider mobile viewport such as 430px.
  • Confirmed the search box has top spacing and is vertically centered.
  • Confirmed the mobile navigation remains usable.
  • Confirmed search focus and dropdown behavior still work.
  • Confirmed the desktop header remains unchanged.

@varma1221 varma1221 self-assigned this Aug 30, 2026
@varma1221 varma1221 added the bug Something isn't working label Aug 30, 2026
@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
by-company Ready Ready Preview Aug 30, 2026 4:45am

@varma1221

Copy link
Copy Markdown
Collaborator Author

@Vardhan-IMMIDI, I’ve pushed the fix for this issue and all checks passed. When you have a moment, could you please review the changes and let me know if everything looks good? Your feedback would be greatly appreciated.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The changes are a small, well-scoped CSS adjustment that directly matches the described mobile layout fix without introducing obvious regressions in the surrounding header rules.

Pull request overview

This PR addresses issue #16 by adjusting the mobile header grid and spacing so the search box is no longer flush against the top of the viewport and is vertically centered within its row on narrow screens.

Changes:

  • Defines an explicit two-row grid for the mobile header container and adds vertical padding.
  • Pins the search container to the first grid row and vertically centers it.
  • Keeps the mobile nav in the second row without affecting the desktop header layout.
File summaries
File Description
site/app/globals.css Updates the @media (max-width: 760px) header grid to add row structure, padding, and explicit search placement/alignment for mobile.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Vardhan-IMMIDI Vardhan-IMMIDI left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! @varma1221. Props on a really clean PR

@Vardhan-IMMIDI
Vardhan-IMMIDI merged commit 16d6987 into main Aug 30, 2026
4 checks passed
@Vardhan-IMMIDI
Vardhan-IMMIDI deleted the fix/mobile-header-search-alignment branch August 30, 2026 05:09

This branch was successfully deployed

1 active deployment
Preview — 82b9423c Deployed Aug 30, 2026 by vercel[bot]
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.

Mobile UI: Search box alignment issue in header

3 participants