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

Cleanup: Dialog.jsx is unused (no component imports it) #115

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

Problem

src/components/ui/Dialog.jsx is a Radix Dialog component that is never imported by any page or component. It adds ~2KB to the bundle and increases maintenance surface.

The project uses Sheet (also Radix Dialog-based) for the mobile menu, but never uses a standalone Dialog/modal.

Fix

Delete src/components/ui/Dialog.jsx unless a modal/dialog is planned for the future.

Files

  • src/components/ui/Dialog.jsx — delete

Severity

Low — dead code, minimal bundle impact

## Problem src/components/ui/Dialog.jsx is a Radix Dialog component that is never imported by any page or component. It adds ~2KB to the bundle and increases maintenance surface. The project uses Sheet (also Radix Dialog-based) for the mobile menu, but never uses a standalone Dialog/modal. ## Fix Delete src/components/ui/Dialog.jsx unless a modal/dialog is planned for the future. ## Files - src/components/ui/Dialog.jsx — delete ## Severity Low — dead code, minimal bundle impact
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#115
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?