fix: Duplicate navigation on home and profile pages #26

Closed
opened 2026-04-09 21:21:55 +00:00 by KnallerKalle · 1 comment
KnallerKalle commented 2026-04-09 21:21:55 +00:00 (Migrated from github.com)

Problem

The home page (page.tsx) and profile page both import the old Navigation component, while the layout already provides MobileNavigation for ALL pages. This creates:

  1. Duplicate desktop sidebars - Two sidebars render simultaneously
  2. Inconsistent nav items - Navigation has 7 items (incl. Tags, Profile), MobileNavigation has 5
  3. Missing items - MobileNavigation doesn't include Tags, Profile, Analytics, FI pages
  4. Duplicate mobile bars - Both components render mobile bottom navigation

Solution

  1. Remove Navigation import from home and profile pages
  2. Consolidate all nav items into MobileNavigation (the layout-level component)
  3. Add missing pages (Tags, Profile, Analytics, FI) to MobileNavigation
  4. Remove the old Navigation.tsx component entirely

Files Affected

  • src/app/[locale]/page.tsx - Remove Navigation import
  • src/app/[locale]/profile/page.tsx - Remove Navigation import
  • src/components/MobileNavigation.tsx - Add missing nav items
  • src/components/Navigation.tsx - Delete (replaced by MobileNavigation)
## Problem The home page (`page.tsx`) and profile page both import the old `Navigation` component, while the layout already provides `MobileNavigation` for ALL pages. This creates: 1. **Duplicate desktop sidebars** - Two sidebars render simultaneously 2. **Inconsistent nav items** - Navigation has 7 items (incl. Tags, Profile), MobileNavigation has 5 3. **Missing items** - MobileNavigation doesn't include Tags, Profile, Analytics, FI pages 4. **Duplicate mobile bars** - Both components render mobile bottom navigation ## Solution 1. Remove `Navigation` import from home and profile pages 2. Consolidate all nav items into `MobileNavigation` (the layout-level component) 3. Add missing pages (Tags, Profile, Analytics, FI) to MobileNavigation 4. Remove the old `Navigation.tsx` component entirely ## Files Affected - `src/app/[locale]/page.tsx` - Remove Navigation import - `src/app/[locale]/profile/page.tsx` - Remove Navigation import - `src/components/MobileNavigation.tsx` - Add missing nav items - `src/components/Navigation.tsx` - Delete (replaced by MobileNavigation)
KnallerKalle commented 2026-04-09 21:39:30 +00:00 (Migrated from github.com)

Fixed in commit 36f3b12

  • Removed duplicate Navigation imports from home and profile pages
  • Consolidated all navigation into MobileNavigation component (layout-level)
  • Added all nav items: Home, Library, Knowledge, Generate, FI, Tags, Analytics, Settings, Profile
  • Added LanguageSelector to sidebar
  • Build passes, 27/27 tests passing
Fixed in commit 36f3b12 - Removed duplicate Navigation imports from home and profile pages - Consolidated all navigation into MobileNavigation component (layout-level) - Added all nav items: Home, Library, Knowledge, Generate, FI, Tags, Analytics, Settings, Profile - Added LanguageSelector to sidebar - Build passes, 27/27 tests passing
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Kalle/ATM#26
No description provided.