Skip to content

fix(nav): unify mobile navbar surface - #80

Merged
devrodri-com merged 3 commits into
mainfrom
fix/devrodri-mobile-navbar-surface
Aug 7, 2026
Merged

fix(nav): unify mobile navbar surface#80
devrodri-com merged 3 commits into
mainfrom
fix/devrodri-mobile-navbar-surface

Conversation

@devrodri-com

Copy link
Copy Markdown
Owner

What changed

  • removed the mobile menu's redundant opaque-black surface and smaller blur
  • kept the existing bg-black/70, backdrop-blur-xl, and backdrop-saturate-150 surface on the shared navbar container
  • added focused contracts for the single mobile surface and ES/EN desktop/mobile/no-JavaScript navigation parity

Why

The top mobile bar and expanded panel used different backgrounds and backdrop filters. On iPhone this made the expanded navbar look like two separate surfaces. The outer navbar already wraps both regions, so the child panel only needed to stop painting a second surface.

Production and source discovery also confirmed that Servicios/Services was already present exactly once in the shared primary navigation registry. No navigation or copy change was required.

Validation

  • Node 22.23.2; npm 9.9.4
  • npm ci
  • npm run typecheck
  • npm run lint
  • npm run test:run — 31 suites, 330 tests
  • npm run build — prerender and HTTP verification passed
  • npm run ci
  • Chromium and WebKit checks at 390×844, 430×932, 768×1024, and 1440×1000
  • ES/EN routes, mobile open/close, Escape, focus return, touch, reduced motion, no-JavaScript fallback, overflow, and desktop parity

Review gate

  • Keep this PR in Draft.
  • Physical verification on Rodrigo's iPhone is required before marking ready or merging.
  • No production deployment or merge is part of VIS-NAV-01.

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
minisitio-rodrigo Ready Ready Preview Aug 7, 2026 1:07am

Request Review

@devrodri-com devrodri-com reopened this Aug 7, 2026
@devrodri-com
devrodri-com marked this pull request as ready for review August 7, 2026 01:10
@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@devrodri-com
devrodri-com merged commit 9a1de95 into main Aug 7, 2026
7 checks passed
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