Skip to content

fix: improve mobile layout for website and fix docs sidebar on mobile#4379

Merged
NathanFlurry merged 1 commit intomainfrom
mobile-layout-fixes
Mar 8, 2026
Merged

fix: improve mobile layout for website and fix docs sidebar on mobile#4379
NathanFlurry merged 1 commit intomainfrom
mobile-layout-fixes

Conversation

@NathanFlurry
Copy link
Member

Description

Improves mobile layout across the website and fixes the docs sidebar navigation not appearing in the mobile hamburger menu. These changes address viewport issues at 375px width and provide a better mobile experience overall.

Changes

  • Docs sidebar on mobile: The sidebar data is now passed through the DocsLayout to the Header component and rendered in the mobile navigation drawer, allowing users to navigate docs on mobile
  • Section padding: Reduced excessive vertical padding on landing page sections from 192px (py-48) to 48px (py-16) on mobile, only expanding to 192px on desktop
  • Footer layout: Changed footer grid to single-column layout on screens under 440px width
  • Features grid: Removed the three-column breakpoint so the grid only goes 1 → 2 → 4 columns, providing better spacing

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Tested at 375px viewport width:

  • Verified docs sidebar now appears in hamburger menu on mobile
  • Confirmed landing page sections no longer have excessive padding gaps
  • Verified footer stacks properly on narrow screens
  • Checked that Features grid displays correctly with only 1, 2, or 4 column layouts
  • Tested desktop viewports (1024px, 1440px) to ensure no regressions

Co-Authored-By: Claude Haiku 4.5 noreply@anthropic.com

- Fix docs sidebar not appearing in mobile hamburger menu by passing sidebar data through layout and rendering Tree component
- Reduce excessive section padding on mobile (py-16 md:py-48) for 5 marketing sections
- Improve footer mobile layout with single-column grid on narrow screens
- Fix Features section grid to skip three-column layout (1 → 2 → 4 columns only)
- Fix TypeScript error in KV operations by adding missing key label types

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@railway-app railway-app bot temporarily deployed to rivet-frontend / rivet-pr-4379 March 8, 2026 03:43 Destroyed
@railway-app
Copy link

railway-app bot commented Mar 8, 2026

🚅 Deployed to the rivet-pr-4379 environment in rivet-frontend

Service Status Web Updated (UTC)
website ✅ Success (View Logs) Web Mar 8, 2026 at 3:48 am
frontend-cloud ✅ Success (View Logs) Web Mar 8, 2026 at 3:46 am
frontend-inspector ✅ Success (View Logs) Web Mar 8, 2026 at 3:46 am
mcp-hub ✅ Success (View Logs) Web Mar 8, 2026 at 3:43 am
ladle ❌ Build Failed (View Logs) Web Mar 8, 2026 at 3:43 am

@claude
Copy link

claude bot commented Mar 8, 2026

PR Review

Overall this is a clean, well-scoped mobile UX improvement.

Unrelated Change

The kv-limits.ts type extension (adding start key and end key labels) is unrelated to mobile layout fixes and should be in its own PR or at least mentioned in the description. It is a small and correct change, but it makes the diff harder to review in context.

Header / Docs Sidebar on Mobile

The approach of passing sidebarData through DocsLayout -> Header -> DocsMobileNavigation is clean.

  1. Performance: The full SidebarItem array is now serialized into the client-side bundle via client:load on Header. Depending on how large the sitemap can grow, this could add non-trivial bytes to the initial JS payload. Worth verifying the size of foundTab?.tab?.sidebar in the largest docs sections.

  2. Fallback priority: The guard that renders sidebarData only when no tree slot is provided is correct — good defensive behavior.

  3. Type improvement: Explicitly typing DocsMobileNavigation is strictly better than the previous implicit any.

Section Padding

The py-48 to py-16 md:py-48 change is applied consistently across HostingSection, IntegrationsSection, ObservabilitySection, RedesignedCTA, and SolutionsSection. If other landing page sections also use py-48 but are not covered here, they should be updated for consistency.

BuiltInFeatures Grid

Removing sm:grid-cols-3 means the grid jumps from 2 columns directly to 4 columns at lg. On screens between 640px and 1024px features render in 2 columns. This is a design call, but worth double-checking that 8 items in a 2-column grid does not look cramped in that range.

Footer

The custom breakpoint min-[440px] is consistent across Footer.jsx and BuiltInFeatures.tsx. Using items-start on mobile then centering at 440px for the investor badges row is a nice touch.


No blocking issues. The mobile sidebar fix is the meaningful change here and the implementation looks correct.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 8, 2026

More templates

@rivetkit/cloudflare-workers

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/cloudflare-workers@4379

@rivetkit/framework-base

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/framework-base@4379

@rivetkit/next-js

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/next-js@4379

@rivetkit/react

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/react@4379

rivetkit

pnpm add https://pkg.pr.new/rivet-dev/rivet/rivetkit@4379

@rivetkit/sql-loader

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sql-loader@4379

@rivetkit/sqlite-vfs

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sqlite-vfs@4379

@rivetkit/traces

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/traces@4379

@rivetkit/workflow-engine

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/workflow-engine@4379

@rivetkit/virtual-websocket

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/virtual-websocket@4379

@rivetkit/engine-runner

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner@4379

@rivetkit/engine-runner-protocol

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner-protocol@4379

commit: 1d0d303

@NathanFlurry NathanFlurry merged commit b67c07d into main Mar 8, 2026
11 of 22 checks passed
@NathanFlurry NathanFlurry deleted the mobile-layout-fixes branch March 8, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant