Skip to content

[#104] Add persistent global navigation bar#108

Merged
realproject7 merged 1 commit intomainfrom
task/104-global-navbar
Mar 15, 2026
Merged

[#104] Add persistent global navigation bar#108
realproject7 merged 1 commit intomainfrom
task/104-global-navbar

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

Fixes #104

  • NavBar component (src/components/NavBar.tsx): Fixed top bar with terminal aesthetic — $ PlotLink logo, nav links (discover, create, writer, reader, chain), active page indicator (>), backdrop blur
  • Mobile responsive: Hamburger toggle ([=]/[x]) with dropdown menu showing all links + wallet connect
  • Root layout: NavBar added to layout.tsx with pt-11 offset for fixed positioning
  • ConnectWallet consolidated: Removed duplicate ConnectWallet from 5 pages (home, create, writer dashboard, reader dashboard, chain) — now lives exclusively in the nav bar

Test plan

  • Nav bar visible on all pages
  • Active link highlighted with > prefix and accent background
  • ConnectWallet works from nav bar on all pages
  • Mobile: hamburger opens/closes menu, links navigate and close menu
  • No duplicate ConnectWallet buttons on any page

🤖 Generated with Claude Code

- Created NavBar component with terminal aesthetic: $ PlotLink logo,
  discover/create/writer/reader/chain links, active state indicator (>),
  mobile hamburger menu ([=]/[x])
- Added NavBar to root layout.tsx with pt-11 content offset
- Moved ConnectWallet into nav bar, removed from all individual pages
  (page.tsx, create, writer dashboard, reader dashboard, chain)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T2b Review — APPROVE

Clean global nav implementation:

  • Terminal aesthetic ($ PlotLink, > active indicator, [=]/[x] hamburger) ✓
  • ConnectWallet consolidated into nav, removed from 5 pages ✓
  • Mobile responsive dropdown with auto-close on navigate ✓
  • Active state uses startsWith for nested route matching ✓
  • pt-11 offset in layout for fixed nav height ✓

LGTM ✓

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

This adds the persistent global nav in the root layout, consolidates wallet connection into a single shared location, and keeps the UI responsive with a mobile menu and active-state styling.

Findings

  • No blocking findings.

Decision

Approving because the nav is integrated at the layout level, duplicate ConnectWallet usages were removed from the touched pages, and the refreshed lint/typecheck run passed.

@realproject7 realproject7 merged commit 9df7c91 into main Mar 15, 2026
1 check 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.

[P4.5-1] Global navigation bar

2 participants