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

SEO/a11y: Navigation uses Button onClick instead of Link elements #102

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

Problem

Throughout the site, internal navigation uses Button onClick={navigate(/path)} instead of Link to=/path. This happens in:

  • Home.jsx: Hero CTAs, service cards, industry cards, trust section CTAs
  • ServiceDetail.jsx: Request This Service button
  • About.jsx: Request Consultation button

Why this matters:

  1. Accessibility: Screen readers announce these as buttons, not links. Users cannot right-click to open in new tab.
  2. SEO: Search crawlers cannot follow button onClick - they need a href or Link elements.
  3. UX: Users cannot Cmd+Click or Ctrl+Click to open in new tab.

Some pages (Contact.jsx, About.jsx) already use Link correctly. Make it consistent.

Fix

Replace all internal navigation Button onClick={navigate} with Link components styled as buttons.

Files

  • src/pages/Home.jsx - 6+ instances
  • src/pages/ServiceDetail.jsx - sidebar CTA
  • src/pages/About.jsx - CTA section

Severity

Medium - impacts SEO crawlability and accessibility

## Problem Throughout the site, internal navigation uses Button onClick={navigate(/path)} instead of Link to=/path. This happens in: - Home.jsx: Hero CTAs, service cards, industry cards, trust section CTAs - ServiceDetail.jsx: Request This Service button - About.jsx: Request Consultation button Why this matters: 1. Accessibility: Screen readers announce these as buttons, not links. Users cannot right-click to open in new tab. 2. SEO: Search crawlers cannot follow button onClick - they need a href or Link elements. 3. UX: Users cannot Cmd+Click or Ctrl+Click to open in new tab. Some pages (Contact.jsx, About.jsx) already use Link correctly. Make it consistent. ## Fix Replace all internal navigation Button onClick={navigate} with Link components styled as buttons. ## Files - src/pages/Home.jsx - 6+ instances - src/pages/ServiceDetail.jsx - sidebar CTA - src/pages/About.jsx - CTA section ## Severity Medium - impacts SEO crawlability and accessibility
null closed this issue 2026-05-17 21:45:09 -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#102
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?