The new design shipped and is the default, but the "New design" switch in Settings → Appearance and the old classic look were still in the code. Every redesigned element carried two styles, which made the UI harder to change. Turning the switch off showed a half-maintained old look.
The new design is now the only design. Users see no change if they never turned the switch off, which was the default.
redesign: Tailwind variant and the data-redesign attribute are gone. Every redesign: class is now a plain class.html[data-redesign] CSS rules are unconditional, with the same cascade.pierre-light syntax theme is dropped, so light code always uses the readable palette.theme-color, and the desktop window background used the old #0a0a0a dark canvas, so dark mode flashed gray before the app's true black. They now use #000.Settings → Appearance → Interface. The only visible change.
| main | this PR |
|---|---|
| main settings | this PR settings |
Everything else renders the same as main with the new design on:
| Dark | Light |
|---|---|
| thread dark | thread light |
Proof:
redesign: prefix changed a cn() merge: the model picker's selected marker and the font preview height. Both are fixed to match main.vp run check:changed passes, and the related unit and browser tests pass.Reviewed with sol-loop: 1 round with GPT-6.1 Sol. Sol verdict: approved
Made with Claude Opus 5.5 in Claude Code, running in T3 Code.
🤖 Generated with Claude Code
Added 2 commits
Verdict: Needs Human Approval
Macroscope's review found this PR requires human approval — This PR makes the redesigned web UI unconditional, removes the classic-design setting, and changes behavior across settings, layout, sidebar, composer, diffs, and usage surfaces. The removal of an existing product choice and the resulting default change warrant human review.
You can add or adjust custom eligibility rules. Learn more.
✅ Thread transfer remains within every enforced ceiling.
| Provider | Metric | Main baseline | This PR | Impact | PR ceiling | |
|---|---|---|---|---|---|---|
| Codex | Total thread wire | 4.9 KiB | 5.0 KiB | +30 B (+0.6%) | 6.8 KiB | ✅ |
| Codex | Thread snapshot wire | 3.8 KiB | 3.8 KiB | 0 B (0.0%) | 4.9 KiB | ✅ |
| Codex | Live turn WebSocket wire | 1.2 KiB | 1.2 KiB | +30 B (+2.5%) | 2.0 KiB | ✅ |
| Codex | Live turn WebSocket decoded | 20.8 KiB | 20.9 KiB | +41 B (+0.2%) | 29.3 KiB | ✅ |
| Codex | Live turn messages | 1 | 2 | +1 (+100.0%) | 8 | ✅ |
| Claude | Total thread wire | 5.0 KiB | 5.0 KiB | −24 B (−0.5%) | 6.8 KiB | ✅ |
| Claude | Thread snapshot wire | 3.8 KiB | 3.8 KiB | 0 B (0.0%) | 4.9 KiB | ✅ |
| Claude | Live turn WebSocket wire | 1.2 KiB | 1.2 KiB | −24 B (−2.0%) | 2.0 KiB | ✅ |
| Claude | Live turn WebSocket decoded | 21.2 KiB | 21.2 KiB | −41 B (−0.2%) | 29.3 KiB | ✅ |
| Claude | Live turn messages | 2 | 1 | −1 (−50.0%) | 8 | ✅ |
Baseline: c294d63 · PR result: 6285690 · Source CI: success
10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.
No actionable comments were generated in the recent review.
1eee8638-fc55-4306-a8ff-3b21a3b29302Reviewing files that changed from the base of the PR and between c294d63b2135771dd74f0a2b57af5b501043a74b and 62856905938535156df804792d7ddaa9b31cbc89.
apps/desktop/src/window/DesktopWindow.tsapps/web/index.htmlapps/web/src/appInit.tsxapps/web/src/components/ChatView.tsxapps/web/src/components/ComposerPromptEditorTiptap.tsxapps/web/src/components/DiffPanel.tsxapps/web/src/components/DiffWorkerPoolProvider.tsxapps/web/src/components/GitActionsControl.tsxapps/web/src/components/Sidebar.tsxapps/web/src/components/ThreadStatusIndicators.tsxapps/web/src/components/chat/ChatCanvas.tsxapps/web/src/components/chat/ChatComposer.tsxapps/web/src/components/chat/ComposerPrimaryActions.tsxapps/web/src/components/chat/ModelListRow.tsxapps/web/src/components/chat/ModelPickerContent.tsxapps/web/src/components/chat/ModelPickerSidebar.tsxapps/web/src/components/chat/ThreadDetailsControl.tsxapps/web/src/components/chat/ThreadDetailsPrRow.tsxapps/web/src/components/chat/ThreadDetailsPrRows.tsxapps/web/src/components/chat/ThreadRelationshipsControl.tsxapps/web/src/components/chat/ThreadStatusLine.tsxapps/web/src/components/chat/TraitsPicker.tsxapps/web/src/components/chat/canvasStore.test.tsapps/web/src/components/chat/canvasStore.tsapps/web/src/components/chat/chatCanvasLayout.test.tsapps/web/src/components/chat/chatCanvasLayout.tsapps/web/src/components/pullRequest/PullRequestGhosts.tsxapps/web/src/components/settings/AppearanceSettingsPanel.tsxapps/web/src/components/settings/ProviderInstanceCard.tsxapps/web/src/components/settings/ProviderModelsSection.tsxapps/web/src/components/settings/ProviderSettingsPanel.tsxapps/web/src/components/settings/SettingsFontPreviews.tsxapps/web/src/components/settings/SettingsPanels.tsxapps/web/src/components/settings/ThemePreviewCircles.tsxapps/web/src/components/settings/ThemeSettings.tsxapps/web/src/components/settings/settingsSearch.tsapps/web/src/components/sidebar/SidebarListMarkers.tsxapps/web/src/components/sidebar/SidebarThreadHeader.tsxapps/web/src/components/sidebar/SidebarThreadRow.tsxapps/web/src/components/ui/button.tsxapps/web/src/components/ui/collapsible-section-header.tsxapps/web/src/components/ui/combobox.tsxapps/web/src/components/ui/command.tsxapps/web/src/components/ui/kbd.tsxapps/web/src/components/usage/UsageLimitsPooled.tsxapps/web/src/components/usage/UsagePage.tsxapps/web/src/index.cssapps/web/src/lib/diffRendering.tsapps/web/src/lib/incrementalHighlighting.tsapps/web/src/lib/syntaxHighlighting.tspackages/contracts/src/settings.tsIncluded review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review.
The pull request removes the New design toggle and its conditional styling paths. It makes the selected chat, sidebar, settings, usage, and diff behavior unconditional, updates chat-canvas positioning, and changes dark-mode backgrounds to pure black.
Changes: Default interface and appearance
| Layer / File(s) | Summary |
|---|---|
Remove the redesign setting and update default appearance packages/contracts/src/settings.ts, apps/web/src/appInit.tsx, apps/web/src/components/settings/AppearanceSettingsPanel.tsx, apps/web/src/components/settings/settingsSearch.ts, apps/web/src/components/settings/SettingsPanels.tsx, apps/web/src/index.css, apps/web/index.html, apps/desktop/src/window/DesktopWindow.ts | The client settings contract and appearance controls no longer include the redesign setting. Startup no longer toggles data-redesign. Dark-mode boot and canvas backgrounds change to black, and the selected global styles apply without the redesign variant. |
Unify chat and model-picker behavior apps/web/src/components/ChatView.tsx, apps/web/src/components/chat/* | The chat canvas resolves its position beside details cards without centerBesideDetails. Chat controls, composer, and model picker no longer branch on the redesign setting. |
Unify sidebar and shared controls apps/web/src/components/Sidebar.tsx, apps/web/src/components/ThreadStatusIndicators.tsx, apps/web/src/components/sidebar/*, apps/web/src/components/ui/*, apps/web/src/components/pullRequest/PullRequestGhosts.tsx | Sidebar rows, section counts, and shared controls use the selected styles and rendering without redesign-specific variants. Slim sidebar rows no longer render the PR badge. |
Unify settings and usage surfaces apps/web/src/components/settings/*, apps/web/src/components/usage/* | Provider and appearance settings use consistent layouts and focus styles. Usage period controls are omitted when Limits is selected, and the pooled percentage is shown only when the account name is shown. |
Update diff theme and controls apps/web/src/components/DiffPanel.tsx, apps/web/src/components/DiffWorkerPoolProvider.tsx, apps/web/src/lib/diffRendering.ts, apps/web/src/lib/incrementalHighlighting.ts, apps/web/src/lib/syntaxHighlighting.ts, apps/web/src/index.css | Diff theme selection no longer checks data-redesign; the light theme uses the readable palette. Diff controls and worker subscriptions use the updated theme behavior and types. |
Priority: External image Low
Estimated code review effort: 3 (Moderate) | ~25 minutes
Change: Refactor
Possibly related issues
Possibly related PRs
Suggested reviewers: juliusmarminge, maria-rcks
62856No actionable issue remains from this review; the change is ready for normal merge checks.
Failed Failed checks: 1 warning
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Description check | Warning | The description explains the problem, implementation, visual impact, screenshots, verification steps, test results, and agent usage. However, it does not follow the required section structure and does… | Organize the description under Problem, Change, Scope and approval, and Verification headings. Add a link to the triaged issue or maintainer discussion, including the explicit approval comment. If no prior approval applies, explain why this… |
| Check name | Status | Explanation |
|---|---|---|
| Title check | Passed | The title clearly and concisely describes the primary change: removing the design toggle and making the new design the only design. |
| Linked Issues check | Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | Passed | Check skipped because no linked issues were found for this pull request. |
Explanation
The description explains the problem, implementation, visual impact, screenshots, verification steps, test results, and agent usage. However, it does not follow the required section structure and does not provide the required Scope and approval information, such as a triaged issue or explicit maintainer approval with a link and approval comment.
Resolution
Organize the description under Problem, Change, Scope and approval, and Verification headings. Add a link to the triaged issue or maintainer discussion, including the explicit approval comment. If no prior approval applies, explain why this change qualifies for the exemption. Keep the existing screenshots and specific verification results under Verification.
Comment @coderabbitai help to get the list of available commands.