fix: navbar on ipad

[HarshMN2345]

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

• Style
  • Refined responsive layout behavior for mobile navigation and header visibility across screen sizes
  • Improved logo grid layout display on medium and large viewports
  • Enhanced overall interface responsiveness at various screen breakpoints

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 317dd5f4-a4f5-43e6-9d72-dd94fb7d276e

📥 Commits

Reviewing files that changed from the base of the PR and between 056f076 and b899e29.

📒 Files selected for processing (3)

• src/lib/components/MobileNav.svelte
• src/lib/layouts/Main.svelte
• src/routes/(marketing)/(components)/logo-list.svelte

---

Walkthrough

This pull request adjusts responsive design breakpoints and layout classes across three components. The mobile navigation and header visibility thresholds shift from lg to xl breakpoints, and the logo grid layout changes from displaying 4 columns at md breakpoint to 6 columns at lg breakpoint. All changes are CSS class modifications with no alterations to component logic or state handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main objective of the changes, which adjust responsive breakpoints to fix navbar display issues on iPad-sized screens.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-navbar-for-ipad

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[greptile-apps]

Greptile Summary

This PR fixes the navbar display on iPad by shifting breakpoints from lg (1024px) to xl (1280px), ensuring iPad landscape views (1024px wide) use the mobile nav/header instead of the partially-rendered desktop nav. The logo grid also gains an intermediate 4-column layout at the md breakpoint before expanding to 6 columns at lg.

• MobileNav.svelte: lg:hidden → xl:hidden so the mobile side-nav stays active through iPad landscape width.
• Main.svelte: Both the mobile header (xl:hidden!) and desktop header (xl:block!) are updated consistently to switch at 1280px.
• logo-list.svelte: Adds md:grid-cols-4 between the existing grid-cols-3 and lg:grid-cols-6 to avoid cramped logos on tablets.

The three changes are coherent and consistently applied. Note that the PR description template was left entirely blank — no description, test plan, or issue links were provided.

Confidence Score: 5/5

Safe to merge — the changes are a small, consistent responsive-breakpoint adjustment with no logic, data, or security implications.

All three changes are straightforward Tailwind class swaps that solve a real visual problem (iPad landscape falls exactly on the old lg boundary). The mobile/desktop header and mobile nav are updated in lockstep, and the logo grid gains a sensible intermediate step. No P0 or P1 issues were found.

No files require special attention.

Important Files Changed

Filename	Overview
src/lib/components/MobileNav.svelte	Changed lg:hidden to xl:hidden so the mobile nav persists up to 1280px, covering iPad landscape (1024px).
src/lib/layouts/Main.svelte	Consistently updated both mobile header (lg:hidden! → xl:hidden!) and desktop header (lg:block! → xl:block!) to match the new breakpoint.
src/routes/(marketing)/(components)/logo-list.svelte	Added intermediate md:grid-cols-4 step before lg:grid-cols-6 to prevent the logo grid from being too dense on tablet-sized screens.

_{Reviews (1): Last reviewed commit: "fix: navbar on ipad" | Re-trigger Greptile}