Skip to content

fix(web): keep the ultrathink spectrum still at rest - #56

Draft
saphid wants to merge 1 commit into
whimsy/electron-base-20260905from
whimsy/calm-spectrum
Draft

saphid wants to merge 1 commit into
whimsy/electron-base-20260905from
whimsy/calm-spectrum

Conversation

@saphid

@saphid saphid commented Sep 5, 2026

Copy link
Copy Markdown
Owner

What Changed

Ultrathink continuously rotated the composer border and icon colors, even while idle. Keep the spectrum as a static treatment and replace it with a system-color outline in forced-colors mode.

Why

The mode stays recognizable while the renderer can rest. Enabling and disabling ultrathink still uses the existing composer state.

UI Changes

Cropped Electron component fixtures with synthetic state, using the actual proposed components. The before fixture reproduces the existing presentation; these clips do not establish backend or provider execution.

Before After
Before spectrum After spectrum

Full spectrum interaction, including trigger and settled state

Short playable video · Before recording

Actual integrated Electron surface:

Integrated Electron spectrum

Validation: 13 focused composer-provider-state tests and web typecheck passed. In the real Electron composer the border reports animation-name: none. Before/after fixture recordings demonstrate activation, deactivation and idle behavior; reduced-motion and forced-color checks passed.

Fresh GPT-5.6 Sol read-only source review found no actionable findings against the four frozen feature changes and coding standard 144fbf46af335d8d18a95c8d4b4f2e9e0207fa2e. The direct claude -p --model claude-opus-5 --effort high --tools Read,Grep,Glob --permission-mode plan review launch exited 1 before model execution because OAuth refresh failed. Cross-provider review therefore did not occur.

This draft targets whimsy/electron-base-20260905 at reviewed revision d7462d29, because the working revision diverges from fork main. Only this concern is in the diff. Web and Electron share the renderer; React Native mobile is unchanged and untested. No packaged Nightly build was installed.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for UI changes
  • I included a video for animation/interaction changes

Built with GPT-6 Astra (integration and spectrum) and GPT-5.6 Sol (bounded feature workers and fresh source review), using the Codex harness in T3 Code.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Sep 5, 2026
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for def33ef.

This comment will update automatically after the next completed run.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant