chore: cleanup

jaaydenh · jaaydenh · commit 81500d0fe99b · 2025-04-30T16:04:55.000Z
diff --git a/site/src/components/Spinner/Spinner.stories.tsx b/site/src/components/Spinner/Spinner.stories.tsx
@@ -6,7 +6,7 @@ const meta: Meta<typeof Spinner> = {
 	title: "components/Spinner",
 	component: Spinner,
 	args: {
-		children: <PlusIcon className="size-icon-md" />,
+		children: <PlusIcon className="size-icon-lg" />,
 	},
 };
 
diff --git a/site/src/components/Spinner/Spinner.tsx b/site/src/components/Spinner/Spinner.tsx
@@ -14,7 +14,7 @@ const leaves = 8;
 const spinnerVariants = cva("", {
 	variants: {
 		size: {
-			lg: "size-icon-md",
+			lg: "size-icon-lg",
 			sm: "size-icon-sm",
 		},
 	},
diff --git a/site/src/pages/CreateWorkspacePage/CreateWorkspacePageViewExperimental.tsx b/site/src/pages/CreateWorkspacePage/CreateWorkspacePageViewExperimental.tsx
@@ -140,17 +140,6 @@ export const CreateWorkspacePageViewExperimental: FC<
 		}
 	}, [error]);
 
-	useEffect(() => {
-		if (form.submitCount > 0 && form.errors) {
-			const fieldId = `${id}-workspace-name`;
-			const el = document.getElementById(fieldId);
-			if (el) {
-				el.scrollIntoView({ behavior: "smooth", block: "center" });
-				(el as HTMLElement).focus?.();
-			}
-		}
-	}, [form.submitCount, form.errors, id]);
-
 	const [presetOptions, setPresetOptions] = useState([
 		{ label: "None", value: "" },
 	]);
@@ -339,34 +328,27 @@ export const CreateWorkspacePageViewExperimental: FC<
 									<Label className="text-sm" htmlFor={`${id}-workspace-name`}>
 										Workspace name
 									</Label>
-									<div className="flex flex-col">
-										<Input
-											id={`${id}-workspace-name`}
-											value={form.values.name}
-											onChange={(e) => {
-												form.setFieldValue("name", e.target.value.trim());
-												resetMutation();
+									<Input
+										id={`${id}-workspace-name`}
+										value={form.values.name}
+										onChange={(e) => {
+											form.setFieldValue("name", e.target.value.trim());
+											resetMutation();
+										}}
+										disabled={creatingWorkspace}
+									/>
+									<div className="flex gap-2 text-xs text-content-secondary items-center">
+										Need a suggestion?
+										<Button
+											variant="subtle"
+											size="sm"
+											onClick={async () => {
+												await form.setFieldValue("name", suggestedName);
+												rerollSuggestedName();
 											}}
-											disabled={creatingWorkspace}
-										/>
-										{form.touched.name && form.errors.name && (
-											<div className="text-content-destructive text-xs mt-2">
-												{form.errors.name}
-											</div>
-										)}
-										<div className="flex gap-2 text-xs text-content-secondary items-center">
-											Need a suggestion?
-											<Button
-												variant="subtle"
-												size="sm"
-												onClick={async () => {
-													await form.setFieldValue("name", suggestedName);
-													rerollSuggestedName();
-												}}
-											>
-												{suggestedName}
-											</Button>
-										</div>
+										>
+											{suggestedName}
+										</Button>
 									</div>
 								</div>
 								{permissions.createWorkspaceForAny && (
diff --git a/site/src/pages/DeploymentSettingsPage/IdpOrgSyncPage/IdpOrgSyncPageView.tsx b/site/src/pages/DeploymentSettingsPage/IdpOrgSyncPage/IdpOrgSyncPageView.tsx
@@ -448,7 +448,7 @@ const OrganizationRow: FC<OrganizationRowProps> = ({
 					aria-label="delete"
 					onClick={() => onDelete(idpOrg)}
 				>
-					<Trash className="size-icon-sm" />
+					<Trash />
 					<span className="sr-only">Delete IdP mapping</span>
 				</Button>
 			</TableCell>
diff --git a/site/src/pages/OrganizationSettingsPage/UserTable/EditRolesButton.tsx b/site/src/pages/OrganizationSettingsPage/UserTable/EditRolesButton.tsx
@@ -135,7 +135,7 @@ export const EditRolesButton: FC<EditRolesButtonProps> = ({
 						size="icon"
 						className="text-content-secondary hover:text-content-primary"
 					>
-						<EditSquare className="size-icon-sm" />
+						<EditSquare />
 					</Button>
 				</Tooltip>
 			</PopoverTrigger>
diff --git a/site/tailwind.config.js b/site/tailwind.config.js
@@ -12,7 +12,7 @@ module.exports = {
 				sans: `"Inter Variable", system-ui, sans-serif`,
 			},
 			size: {
-				"icon-md": "1.5rem",
+				"icon-lg": "1.5rem",
 				"icon-sm": "1.125rem",
 				"icon-xs": "0.875rem",
 			},

Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,7 @@ const OrganizationRow: FC<OrganizationRowProps> = ({`
`448`	`448`	`aria-label="delete"`
`449`	`449`	`onClick={() => onDelete(idpOrg)}`
`450`	`450`	`>`
`451`		`- <Trash className="size-icon-sm" />`
	`451`	`+ <Trash />`
`452`	`452`	`<span className="sr-only">Delete IdP mapping</span>`
`453`	`453`	`</Button>`
`454`	`454`	`</TableCell>`
Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ export const EditRolesButton: FC<EditRolesButtonProps> = ({`
`135`	`135`	`size="icon"`
`136`	`136`	`className="text-content-secondary hover:text-content-primary"`
`137`	`137`	`>`
`138`		`- <EditSquare className="size-icon-sm" />`
	`138`	`+ <EditSquare />`
`139`	`139`	`</Button>`
`140`	`140`	`</Tooltip>`
`141`	`141`	`</PopoverTrigger>`