Skip to content

Fix shared nav on home and claim pages#275

Merged
GsCommand merged 1 commit into
mainfrom
codex/fix-shared-nav-on-home-and-claim-pages
May 22, 2026
Merged

Fix shared nav on home and claim pages#275
GsCommand merged 1 commit into
mainfrom
codex/fix-shared-nav-on-home-and-claim-pages

Conversation

@GsCommand
Copy link
Copy Markdown
Contributor

Motivation

  • The previous change only added a class to the claim page nav and did not adopt the shared container-based nav structure, leaving layout conflicts and inconsistent nav behavior across pages.
  • The intent is to unify the top navigation across pages using the known-good nav from public/protocol.html/public/sdk-records.html and to remove local CSS that conflicts with the shared pattern.

Description

  • Replaced the full <nav>...</nav> block in public/index.html with the shared container-based nav structure used by public/protocol.html/public/sdk-records.html and preserved only Home as the active top-nav item.
  • Replaced the full <nav>...</nav> block in public/claim.html with the same shared nav structure and preserved only Claim as the active top-nav item.
  • Removed the conflicting width/margin rules from the claim page local .nav-inner CSS, leaving only alignment and sizing rules (min-height, padding, display:flex, align-items, justify-content, gap).
  • Verified public/protocol.html was left unchanged and validated that each of public/index.html and public/claim.html contains exactly one <nav> block and the expected active items.

Testing

  • Ran npm test, which executed the test suite and produced 35 test runs with 34 passing and 1 failing test; the failing test is POST /api/auth/verify surfaces dependency unavailable when siwe is missing in tests/api-auth.test.js and appears unrelated to the nav changes. (Test run completed; 34 passed, 1 failed.)
  • Attempted npm run build and confirmed there is no build script in this repository, so a build step was not run.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

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

Project Deployment Actions Updated (UTC)
commandlayer-commandlayer-org Ready Ready Preview, Comment May 22, 2026 6:17pm
commandlayer-org Ready Ready Preview, Comment May 22, 2026 6:17pm
commandlayer-org111 Ready Ready Preview, Comment May 22, 2026 6:17pm

Request Review

@GsCommand GsCommand merged commit 5605105 into main May 22, 2026
4 of 5 checks passed
@GsCommand GsCommand deleted the codex/fix-shared-nav-on-home-and-claim-pages branch May 22, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant