From 71c9b693137c070b4887b365c68659f1a2fb7aed Mon Sep 17 00:00:00 2001 From: Julius Marminge Date: Tue, 22 Sep 2026 21:33:02 -0700 Subject: [PATCH] refactor(web): Button consumers outside the composer stop restyling it (#13208) Co-authored-by: Claude Opus 5.5 (1M context) --- apps/web/src/browser/BrowserDeviceToolbar.tsx | 41 ++++++++---------- apps/web/src/components/ChatMarkdown.tsx | 12 ++---- apps/web/src/components/ChatView.tsx | 2 +- apps/web/src/components/CommandPalette.tsx | 12 ++---- .../web/src/components/CustomSnoozeDialog.tsx | 2 +- apps/web/src/components/DiffPanel.tsx | 13 +++--- apps/web/src/components/GitActionsControl.tsx | 23 ++++------ .../src/components/ProjectScriptsControl.tsx | 42 ++++++++++--------- .../web/src/components/ServerUpdateAction.tsx | 4 +- apps/web/src/components/Sidebar.tsx | 2 +- .../chat/AssistantSelectionToolbar.tsx | 2 +- .../components/chat/ContextWindowMeter.tsx | 2 +- .../components/chat/ExpandedImageDialog.tsx | 13 ++---- .../src/components/chat/MessageCopyButton.tsx | 5 +-- .../src/components/chat/MessagesTimeline.tsx | 4 +- apps/web/src/components/chat/OpenInPicker.tsx | 1 - .../chat/SnapShotAttachmentDetails.tsx | 7 +--- .../diffs/DiffCommentAnnotation.tsx | 19 +++++---- .../components/onboarding/WelcomeWizard.tsx | 12 +++--- .../components/preview/PreviewMoreMenu.tsx | 1 - .../src/components/projectScriptEditor.tsx | 2 +- .../pullRequest/PullRequestCodeTab.tsx | 2 +- .../pullRequest/PullRequestComposer.tsx | 8 +--- .../pullRequest/PullRequestDetailPanel.tsx | 3 +- .../pullRequest/PullRequestEditButton.tsx | 13 +++--- .../PullRequestReviewAnnotation.tsx | 8 +--- .../pullRequest/PullRequestSummaryTab.tsx | 4 +- .../settings/ConnectionsSettings.tsx | 13 +----- .../settings/KeybindingsSettings.tsx | 4 +- .../settings/ProjectActionsList.tsx | 21 +++++----- .../settings/ProviderAccentColorPicker.tsx | 32 +++++++------- .../settings/SettingsSidebarNav.tsx | 4 +- .../sidebar/SidebarProviderUpdatePill.tsx | 10 ++--- .../sidebar/SidebarThreadHeader.tsx | 4 +- apps/web/src/components/ui/button.tsx | 4 +- apps/web/src/components/ui/input.tsx | 3 ++ apps/web/src/index.css | 20 +-------- scripts/lint-restyle-ceiling.ts | 2 +- 38 files changed, 156 insertions(+), 220 deletions(-) diff --git a/apps/web/src/browser/BrowserDeviceToolbar.tsx b/apps/web/src/browser/BrowserDeviceToolbar.tsx index f84e18d8466987ede3ffdeba746fbd7516037121..a8c9027f8a16f359d4fef290c8d5a3da7b43fdd4 100644 --- a/apps/web/src/browser/BrowserDeviceToolbar.tsx +++ b/apps/web/src/browser/BrowserDeviceToolbar.tsx @@ -242,10 +242,7 @@ export function BrowserDeviceToolbar({ onChange={(event) => updateCustomDimension("width", event.target.value)} aria-label="Viewport width" aria-invalid={!customValid} - className={cn( - "text-center [&_[data-slot=input]::-webkit-inner-spin-button]:appearance-none [&_[data-slot=input]]:[appearance:textfield]", - width >= 360 ? "w-14" : "w-13", - )} + className={width >= 360 ? "w-14" : "w-13"} /> × updateCustomDimension("height", event.target.value)} aria-label="Viewport height" aria-invalid={!customValid} - className={cn( - "text-center [&_[data-slot=input]::-webkit-inner-spin-button]:appearance-none [&_[data-slot=input]]:[appearance:textfield]", - width >= 360 ? "w-14" : "w-13", - )} + className={width >= 360 ? "w-14" : "w-13"} /> @@ -281,14 +275,13 @@ export function BrowserDeviceToolbar({ event.preventDefault()} onClick={toggleAspectRatio} @@ -315,19 +308,21 @@ export function BrowserDeviceToolbar({ > - + {/* Sticky backing so scrolled controls do not show through the close action. */} + + + ); } diff --git a/apps/web/src/components/ChatMarkdown.tsx b/apps/web/src/components/ChatMarkdown.tsx index ef833e0e679489d78bb53bb0bdd7bed626bc4a77..f48c573aa8bc0066caf0908083f293c6a732317e 100644 --- a/apps/web/src/components/ChatMarkdown.tsx +++ b/apps/web/src/components/ChatMarkdown.tsx @@ -784,9 +784,8 @@ function MarkdownTable({ children, ...props }: React.ComponentProps<"table">) { render={ + + ); @@ -269,7 +273,7 @@ export default function ProjectScriptsControl({ diff --git a/apps/web/src/components/chat/OpenInPicker.tsx b/apps/web/src/components/chat/OpenInPicker.tsx index 35a75a20a36e47ce613d1049ce942514b3ad70ad..3c0b2cd118ed99506723c5a8c7847b28047617d0 100644 --- a/apps/web/src/components/chat/OpenInPicker.tsx +++ b/apps/web/src/components/chat/OpenInPicker.tsx @@ -353,7 +353,6 @@ export const OpenInPicker = memo(function OpenInPicker({ + + + ) : null} ); diff --git a/apps/web/src/components/onboarding/WelcomeWizard.tsx b/apps/web/src/components/onboarding/WelcomeWizard.tsx index 100869f25369711ee2acdeba183c3a0cb06a9ffe..26ba0cea419df5332e3f93dae4dbf6c9b9f24f10 100644 --- a/apps/web/src/components/onboarding/WelcomeWizard.tsx +++ b/apps/web/src/components/onboarding/WelcomeWizard.tsx @@ -357,12 +357,13 @@ function ConnectionStep({ render={ + + ); } diff --git a/apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx b/apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx index 54f4860b95338a13d13b11b889aa502d9840d2a4..11acd48a4c105df8be4799e9794fe0b06d5d5e9b 100644 --- a/apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx +++ b/apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx @@ -312,7 +312,6 @@ export function ReviewThreadCard({ ) diff --git a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx index c4f3102011829f66acdd180eb9109a1896c97cd7..33b9a42c6ce061641f5f80d3193f8e58496e89de 100644 --- a/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx +++ b/apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx @@ -922,8 +922,8 @@ export function PullRequestSummaryTab({ actions={ diff --git a/apps/web/src/components/settings/KeybindingsSettings.tsx b/apps/web/src/components/settings/KeybindingsSettings.tsx index bc8e4e9df31c9a75e538008b0cce5447930a7913..3512a6a25a6c66143f172ae678566f9fd5f75133 100644 --- a/apps/web/src/components/settings/KeybindingsSettings.tsx +++ b/apps/web/src/components/settings/KeybindingsSettings.tsx @@ -891,12 +891,12 @@ function WhenClauseControl({ + + + } /> diff --git a/apps/web/src/components/settings/ProviderAccentColorPicker.tsx b/apps/web/src/components/settings/ProviderAccentColorPicker.tsx index 22e91a24553d14dd53eee90c7f37b9f8f16eae12..76f6b7bae52cdeae6db69e3332f5b0b882e0aa24 100644 --- a/apps/web/src/components/settings/ProviderAccentColorPicker.tsx +++ b/apps/web/src/components/settings/ProviderAccentColorPicker.tsx @@ -92,21 +92,23 @@ function ProviderCustomColorPicker(props: { /> - - - Clear color - - } - /> +
+ + + Clear color + + } + /> +
); diff --git a/apps/web/src/components/settings/SettingsSidebarNav.tsx b/apps/web/src/components/settings/SettingsSidebarNav.tsx index 5f4d29b00bc4ef1b7832dfe74bea2989133b0cf2..ec46ab1f13a30fb405fcc1c22aefba2d2a65902d 100644 --- a/apps/web/src/components/settings/SettingsSidebarNav.tsx +++ b/apps/web/src/components/settings/SettingsSidebarNav.tsx @@ -265,8 +265,8 @@ export function SettingsSidebarNav({ pathname }: { pathname: string }) { + } /> Dismiss until provider status changes diff --git a/apps/web/src/components/sidebar/SidebarThreadHeader.tsx b/apps/web/src/components/sidebar/SidebarThreadHeader.tsx index 07c1c8e86535bb6da9f177ff62bfeea5931a2d00..839c2ba37f01a53c26ccc1a17a502b772c9d90bb 100644 --- a/apps/web/src/components/sidebar/SidebarThreadHeader.tsx +++ b/apps/web/src/components/sidebar/SidebarThreadHeader.tsx @@ -108,8 +108,8 @@ export function SidebarThreadHeader({