authorgravatar for julius0216@outlook.comJulius Marminge <julius0216@outlook.com> 2026-09-22 12:22:12-07:00
committergravatar for noreply@github.comGitHub <noreply@github.com> 2026-09-22 12:22:12-07:00
log9515016166f926968dacaefc69e2ea0af63942ae
tree450cf200cf4f38b053359192d99c24ecd7dbb4d5
parentbbd5cc03231b8b831b44bdbd6e62bf736391cb1f
signature Signed by PGP key B5690EEEBB952194

refactor(web): use icon-xs where icon buttons were forced to size-6 (#13025)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

8 files changed, 8 insertions(+), 18 deletions(-)

apps/web/src/components/LegacySidebar.tsx+1-1
...@@ -3040,7 +3040,7 @@ const SidebarProjectsContent = memo(function SidebarProjectsContent(...@@ -3040,7 +3040,7 @@ const SidebarProjectsContent = memo(function SidebarProjectsContent(
3040 variant="ghost-muted"3040 variant="ghost-muted"
3041 aria-label="Add project"3041 aria-label="Add project"
3042 data-testid="sidebar-add-project-trigger"3042 data-testid="sidebar-add-project-trigger"
3043 className="size-6 [--control-icon-color:currentColor] text-icon-muted"3043 className="[--control-icon-color:currentColor] text-icon-muted"
3044 onClick={openAddProject}3044 onClick={openAddProject}
3045 />3045 />
3046 }3046 }
apps/web/src/components/RightPanelTabs.tsx+1-1
...@@ -1260,7 +1260,7 @@ export function RightPanelTabs(props: RightPanelTabsProps) {...@@ -1260,7 +1260,7 @@ export function RightPanelTabs(props: RightPanelTabsProps) {
1260 render={1260 render={
1261 <Button1261 <Button
1262 aria-label="Add panel surface"1262 aria-label="Add panel surface"
1263 className="size-6 shrink-0"1263 className="shrink-0"
1264 size="icon-xs"1264 size="icon-xs"
1265 variant="ghost-muted"1265 variant="ghost-muted"
1266 />1266 />
apps/web/src/components/Sidebar.tsx+1-1
...@@ -4517,7 +4517,7 @@ export default function Sidebar() {...@@ -4517,7 +4517,7 @@ export default function Sidebar() {
4517 tabIndex={-1}4517 tabIndex={-1}
4518 aria-hidden="true"4518 aria-hidden="true"
4519 title={`Project settings for ${project.displayName}`}4519 title={`Project settings for ${project.displayName}`}
4520 className="ml-auto size-6 [--control-icon-color:currentColor] text-icon-muted focus-visible:bg-accent focus-visible:text-foreground"4520 className="ml-auto [--control-icon-color:currentColor] text-icon-muted focus-visible:bg-accent focus-visible:text-foreground"
4521 onPointerDown={(event) => event.stopPropagation()}4521 onPointerDown={(event) => event.stopPropagation()}
4522 onClick={(event) => {4522 onClick={(event) => {
4523 void handleProjectSettings(event, project);4523 void handleProjectSettings(event, project);
apps/web/src/components/chat/MessagesTimeline.tsx+2-4
...@@ -1814,9 +1814,8 @@ function QueuedMessageTimelineRow({...@@ -1814,9 +1814,8 @@ function QueuedMessageTimelineRow({
1814 render={1814 render={
1815 <Button1815 <Button
1816 type="button"1816 type="button"
1817 size="icon-micro"1817 size="icon-xs"
1818 variant="ghost-muted"1818 variant="ghost-muted"
1819 className="size-6"
1820 onPointerDown={(event) => event.preventDefault()}1819 onPointerDown={(event) => event.preventDefault()}
1821 onClick={() => ctx.onSteerQueuedMessage(queuedMessage.id)}1820 onClick={() => ctx.onSteerQueuedMessage(queuedMessage.id)}
1822 aria-label="Send now"1821 aria-label="Send now"
...@@ -1837,9 +1836,8 @@ function QueuedMessageTimelineRow({...@@ -1837,9 +1836,8 @@ function QueuedMessageTimelineRow({
1837 render={1836 render={
1838 <Button1837 <Button
1839 type="button"1838 type="button"
1840 size="icon-micro"1839 size="icon-xs"
1841 variant="ghost-muted"1840 variant="ghost-muted"
1842 className="size-6"
1843 onPointerDown={(event) => event.preventDefault()}1841 onPointerDown={(event) => event.preventDefault()}
1844 onClick={() => ctx.onRemoveQueuedMessage(queuedMessage.id)}1842 onClick={() => ctx.onRemoveQueuedMessage(queuedMessage.id)}
1845 aria-label="Cancel and return to the composer"1843 aria-label="Cancel and return to the composer"
apps/web/src/components/clerk/ClerkUserProfilePage.tsx+1-1
...@@ -52,7 +52,7 @@ export function ClerkUserProfileRefreshButton({...@@ -52,7 +52,7 @@ export function ClerkUserProfileRefreshButton({
52 <Button52 <Button
53 size="sm"53 size="sm"
54 variant="outline"54 variant="outline"
55 className={cn("text-[0.8125rem]", className)}55 className={className}
56 disabled={disabled || isPending}56 disabled={disabled || isPending}
57 onClick={onClick}57 onClick={onClick}
58 >58 >
apps/web/src/components/clerk/T3ConnectUserProfilePage.tsx+1-8
...@@ -60,12 +60,7 @@ export function T3ConnectEnvironmentRow(props: {...@@ -60,12 +60,7 @@ export function T3ConnectEnvironmentRow(props: {
60 </div>60 </div>
61 <CollapsibleTrigger61 <CollapsibleTrigger
62 render={62 render={
63 <Button63 <Button size="sm" variant="destructive-outline" disabled={props.mutationPending}>
64 size="sm"
65 variant="destructive-outline"
66 className="text-[0.8125rem]"
67 disabled={props.mutationPending}
68 >
69 Deregister64 Deregister
70 </Button>65 </Button>
71 }66 }
...@@ -93,7 +88,6 @@ export function T3ConnectEnvironmentRow(props: {...@@ -93,7 +88,6 @@ export function T3ConnectEnvironmentRow(props: {
93 <Button88 <Button
94 size="sm"89 size="sm"
95 variant="ghost"90 variant="ghost"
96 className="text-[0.8125rem]"
97 disabled={props.mutationPending}91 disabled={props.mutationPending}
98 onClick={() => props.onConfirmationChange(false)}92 onClick={() => props.onConfirmationChange(false)}
99 >93 >
...@@ -102,7 +96,6 @@ export function T3ConnectEnvironmentRow(props: {...@@ -102,7 +96,6 @@ export function T3ConnectEnvironmentRow(props: {
102 <Button96 <Button
103 size="sm"97 size="sm"
104 variant="destructive"98 variant="destructive"
105 className="text-[0.8125rem]"
106 disabled={props.mutationPending}99 disabled={props.mutationPending}
107 onClick={() => props.onDeregister(environment)}100 onClick={() => props.onDeregister(environment)}
108 >101 >
apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx-1
...@@ -2000,7 +2000,6 @@ export function PullRequestDetailPanel({...@@ -2000,7 +2000,6 @@ export function PullRequestDetailPanel({
2000 aria-label={2000 aria-label={
2001 refreshing ? "Refreshing pull request" : "More pull request actions"2001 refreshing ? "Refreshing pull request" : "More pull request actions"
2002 }2002 }
2003 className="size-6"
2004 size="icon-xs"2003 size="icon-xs"
2005 variant="ghost-muted"2004 variant="ghost-muted"
2006 />2005 />
scripts/lint-restyle-ceiling.ts+1-1
...@@ -8,7 +8,7 @@ import * as NodeURL from "node:url";...@@ -8,7 +8,7 @@ import * as NodeURL from "node:url";
8// components/ui exports are migrated to variants (see vite.config.ts). This gate keeps the8// components/ui exports are migrated to variants (see vite.config.ts). This gate keeps the
9// count from growing: CI fails when findings exceed the ceiling. Lower the ceiling when you9// count from growing: CI fails when findings exceed the ceiling. Lower the ceiling when you
10// migrate a file, and delete this script when the rule becomes an error.10// migrate a file, and delete this script when the rule becomes an error.
11export const RESTYLE_CEILING = 1074;11export const RESTYLE_CEILING = 1070;
1212
13const RULE = "shadcn(no-restyle)";13const RULE = "shadcn(no-restyle)";
1414