null/Queue-North-Website
null
/
Queue-North-Website
1
0
Fork 0
Code Issues 27 Pull Requests Packages Projects Releases Wiki Activity

Cleanup: MobileNav.jsx is dead code (Header has its own mobile menu) #113

New Issue
Closed
opened 2026-05-17 21:23:13 -05:00 by null · 0 comments
null commented 2026-05-17 21:23:13 -05:00
Owner
Copy Link

Problem

MobileNav.jsx is a standalone mobile navigation component that is never imported or used anywhere. The Header component already contains its own mobile Sheet menu with the same navigation links.

Both MobileNav and the Header mobile menu duplicate the same nav links, serviceLinks, and industryLinks arrays — meaning any navigation update must be made in two places.

Fix

Delete MobileNav.jsx and remove its export from layout/index.jsx.

Files

  • src/components/layout/MobileNav.jsx — delete
  • src/components/layout/index.jsx — remove export line

Severity

Low — dead code, but maintenance risk from duplication

## Problem MobileNav.jsx is a standalone mobile navigation component that is never imported or used anywhere. The Header component already contains its own mobile Sheet menu with the same navigation links. Both MobileNav and the Header mobile menu duplicate the same nav links, serviceLinks, and industryLinks arrays — meaning any navigation update must be made in two places. ## Fix Delete MobileNav.jsx and remove its export from layout/index.jsx. ## Files - src/components/layout/MobileNav.jsx — delete - src/components/layout/index.jsx — remove export line ## Severity Low — dead code, but maintenance risk from duplication
null closed this issue 2026-05-17 22:08:53 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
dev
main
v0.8.0
v0.2.0
v0.1.1
v0.1.0
Labels
Clear labels   
P0 Critical

Critical — must fix before production. Data loss, security, or outage risk.

  
P1 High

High — should fix soon. Functional bugs, security hardening, data integrity.

  
P2 Medium

Medium — fix in next cycle. Quality of life, robustness, minor security.

  
P3 Low

Low — nice to have. Polish, optimization, minor UX.

  
accessibility

Accessibility issues

  
backend

Server-side code, API, database.

  
bug

Something is broken or behaves incorrectly.

  
content

   
data-integrity

Data consistency, constraints, validation.

  
enhancement

Feature or improvement request

  
frontend

Client-side code, UI components.

  
infra

Docker, deployment, CI/CD, infrastructure.

  
integration

   
owner

Direct feedback from the site owner. Top priority.

  
owner-input

   
performance

Performance or optimization.

  
performance

   
phase-7

   
phase-8

Phase 8 — Deferred enhancements (testimonials, blog, SEO, performance)

  
security

Security vulnerability or hardening.

  
seo

   
ui

UI/visual issues

  
ux

User experience issue.

No Label
P0 Critical
P1 High
P2 Medium
P3 Low
accessibility
backend
bug
content
data-integrity
enhancement
frontend
infra
integration
owner
owner-input
performance
performance
phase-7
phase-8
security
seo
ui
ux
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
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: null/Queue-North-Website#113
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?