From f1a22457854d53eb0aaffe65f4619345ea5c36fa Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 18 May 2026 23:44:17 +0000 Subject: [PATCH] =?UTF-8?q?chore(ui):=20D.10=20cleanup=20=E2=80=94=20drop?= =?UTF-8?q?=20unused=20.ableton-{panel,device,header}?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR 14 of 16 in the UI consistency overhaul. Final cleanup pass. Removes three CSS utility classes from index.css: - .ableton-panel - .ableton-device - .ableton-header All three were retired by D.* migrations — every former consumer now uses the canonical primitives from components/ui/ (Panel, DeviceRack, SectionHeader). `grep -rn` against apps/ui/src/ confirms zero remaining usages. .ableton-shell and .ableton-toolbar STAY because App.tsx:952,956 still consumes them for the outer app chrome (the load-bearing section-grid shell). A focused follow-up could migrate those two to DeviceRack / a future AppShell primitive; out of scope here. Deferred (preserved as-is): - components/MeasurementPrimitives.tsx cannot be fully deleted yet — 5 consumers still depend on timeline/lane primitives (LaneContainer, LaneRow, StatsBar, TimeRuler) that the plan explicitly preserved. The shipped primitives (StatusBadge, DeltaBadge, MetricBar, MetricBarRow, OutlinePillButton, TokenBadgeList, StyledDataTable, AccentMetricCard) also still have consumers in MeasurementDashboard and AnalysisResults sections that weren't migrated (StructureLanes, HarmonyLanes, AnalysisResults remaining sections). Keep the file intact. - components/Tooltip.tsx stays as a re-export shim; the file isn't causing any drift and external imports may still target it. Verified: lint clean, 628/628 unit tests passing, build clean. D.* migrations summary (all merged or open at time of this PR): - D.0 Foundation (#64) ✓ - D.1 Primitives complete (#65) ✓ - D.2 SamplePlayback (#66) ✓ - D.3 SignalChain marquee (#67) ✓ - D.4 FileUpload + Input Source (#69) ✓ - D.5a Results header + metric strip (#71) ✓ - D.5b Mix Chain (#73) ✓ - D.5c Patches (#74) ✓ - D.5d Section headers (#75) ✓ - D.6a Dashboard structure (#76) ✓ - D.6b Dashboard sections (#77) ✓ - D.6c Dashboard headers (#78) ✓ - D.7 SessionMusicianPanel + MixDoctorPanel (#79) — open - D.8 Signal Monitor + SpectrogramViewer drift (#80) — open - D.9 DenseDawConcept drift (#81) — open - D.10 Cleanup (this PR) ✓ https://claude.ai/code/session_01WNtYcWXwn4mVnnjxqT7uAe --- apps/ui/src/index.css | 21 +++++---------------- 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/apps/ui/src/index.css b/apps/ui/src/index.css index 8176c79e..3e271571 100644 --- a/apps/ui/src/index.css +++ b/apps/ui/src/index.css @@ -213,22 +213,11 @@ body { font-variant-numeric: tabular-nums; } -.ableton-panel { - background-color: var(--color-bg-panel); - border: 1px solid var(--color-border); - border-radius: 2px; -} - -.ableton-device { - background-color: var(--color-bg-card); - border: 1px solid var(--color-border); - border-radius: 2px; -} - -.ableton-header { - background-color: var(--color-bg-surface-dark); - border-bottom: 1px solid var(--color-border); -} +/* `.ableton-panel`, `.ableton-device`, and `.ableton-header` were retired + by the D.* migration — every consumer now uses the canonical primitives + from components/ui/ (Panel, DeviceRack, SectionHeader). `.ableton-shell` + and `.ableton-toolbar` stay because App.tsx:952,956 still consumes them + for the outer app chrome. */ .ableton-shell { background-color: var(--color-bg-panel);