Conversation
… CSS hook Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ote + toast), empty state teaches Fn slots
…le save bar is shown
…d :has, typed color rows Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lands the ten deliberately-deferred P3 items from the 2026-06-11 critique (local snapshot
.impeccable/critique/2026-06-11T05-34-28Z__web-src.md, 29/40), plus user-checkpoint decisions.What landed
web/public/dualsense/focus/(20 HUD-era PNGs, 156KB; zero references in web/ or crates/) and the dead.dm-controller-focusCSS. Release file count 60 → 40.--ink-muted-raisedtoken (#93939d, ≈4.94:1 on--surface-raised) applied to the rail's muted text and the discard button (the element actually on the raised surface, 4.46:1 before).friendlyEdgeSlotsErrorat both catch sites so the toast and inline note agree — the raw API string is in a forbidden file). Empty state teaches Fn slots.kind: 'color'fromsavedDiff.ts(no regex sniffing at render time)..work-and-rail:has(.saved-mobile-bar)reserves 84px under the canvas only while the save bar is shown.$:smells — stale telemetry now reads as 0 on the live trigger readouts (gated on the existingselectedGameTelemetryFresh, consistent with the telemetry chip).profileConfigDirtydouble-build deferred: no cheap fix without a runes migration (memoizingbuildControllerConfigmeans hand-listing ~30 deps).Checkpoint
Single batched eyes-on review in dev:mock: Option B chosen for Status, all visual items approved, toast full-border opted in.
Code review notes (deliberate deferrals)
l2/r2LivePresshas a pre-existing hidden dep (legacy$:doesn't tracktelemetryByNameread insidetelemetryUnitValue()); unchanged by this PR, noted for the runes migration.calc(100dvh - 130px)hardcodes measured chrome; revisit if the toolbar's height changes.--ink-muted-raisedis a deliberate fork, not a global--ink-mutedbump (a global bump would touch ~135 uses sight-unseen)..dm-warning/.dm-support-panelstill use side-stripe borders; only the toast was in scope per the user's opt-in..dm-controller-focus-cardCSS family left for a future dead-CSS sweep.Gates (
npm run check: typecheck, source-audit, button-map, snapshot-map, haptics-graph, build, release-size, visual-smoke at 1366/1440/390) green at every commit.🤖 Generated with Claude Code