Repository navigation
Conversation
In use the frequency at the cursor is by far the most important number in the panel, but the readout column set time above it at the same size, so the two competed for the eye and neither was the first read. Frequency is now the only large, lit reading. Time sits immediately beneath it at half the size, captioned, in the same grey as the panel's labels, and the doppler speed at the foot takes that same second rank rather than staying in ink at 19px — two secondary rows in different colours would have read as a third rank. The hairline between the two large readings goes with the second large reading. The LEDs keep their accessible names and classes, so no locator changes. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CrHtVussSR8ZxsxisaepJM
Contributor
PR PreviewPreview deployment is ready! View Preview Last updated: 2026-09-18T08:54:59.646Z |
The fast lane on this PR fails on four cases in tests/unit/multi-tab-merge.test.js, exactly as the base branch does: they left `now` to the wall clock while asserting on tombstones dated 2026-09-05, so they have failed on every run since the seven-day TTL passed. Not this PR's change, and not the design branch's either; main already carries the fix in 9147c49 (PR #333), which this commit ports verbatim. It no-ops once the design branch takes main. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CrHtVussSR8ZxsxisaepJM
Contributor
Author
|
CI: the "Hygiene, lint, typecheck, unit tests, build" check failed on 696ce65 on four cases in Generated by Claude Code |
IanMayo
merged commit Sep 18, 2026
bb1f23f
into
claude/control-design-implementation-9a2bt9
6 checks passed
Merged
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.
Stacked on #326: a change to the readout column asked for after demonstrating the control panel to a stakeholder.
The problem
In use, the frequency at the cursor is by far the most important number in the panel. It was already large, but time was set directly above it at the same size and with the same bloom, so the two competed for the eye and neither was the first read.
The change
TIME, in the same grey as the panel's labels (--gf-muted) rather than in ink, with no bloom.Nothing else moves. The LEDs keep their accessible names and classes, so every locator and the helper's
getLEDValueare unchanged;docs/Control-Panel.md§3 and theCLAUDE.mdentry are updated.Checks
yarn typecheck,yarn lint,yarn hygiene— cleantests/style-panel,control-row,analysis-mode,selection-highlight,player-transport— 47 passedyarn test:unit— 294 passed, 4 failed inmulti-tab-merge.test.js. Those fail identically on the base branch without this change: the fixtures' clock is unpinned, and main already carries the fix (Grey-shades colour map: a live toggle on the player bar and acolour-mapconfig row #333). They will clear when Rebuild the control panel to the design handoff #326 takes main.🤖 Generated with Claude Code
https://claude.ai/code/session_01CrHtVussSR8ZxsxisaepJM
Generated by Claude Code