Repository navigation
fix: align mobile header search box - #17
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@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. |
There was a problem hiding this comment.
🟢 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
left a comment
There was a problem hiding this comment.
Looks good! @varma1221. Props on a really clean PR
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
Verification
npm run lintnpm run buildgit diff --check