Fixes

BrunoQuaresma · BrunoQuaresma · commit ddcb1e9d5142 · 2025-05-01T18:21:08.000Z
diff --git a/site/src/pages/WorkspacePage/Workspace.stories.tsx b/site/src/pages/WorkspacePage/Workspace.stories.tsx
@@ -3,7 +3,7 @@ import type { Meta, StoryObj } from "@storybook/react";
 import type { ProvisionerJobLog } from "api/typesGenerated";
 import { ProxyContext, getPreferredProxy } from "contexts/ProxyContext";
 import * as Mocks from "testHelpers/entities";
-import { withDashboardProvider } from "testHelpers/storybook";
+import { withAuthProvider, withDashboardProvider } from "testHelpers/storybook";
 import { Workspace } from "./Workspace";
 import type { WorkspacePermissions } from "./permissions";
 
@@ -40,8 +40,10 @@ const meta: Meta<typeof Workspace> = {
 				data: Mocks.MockListeningPortsResponse,
 			},
 		],
+		user: Mocks.MockUser,
 	},
 	decorators: [
+		withAuthProvider,
 		withDashboardProvider,
 		(Story) => (
 			<ProxyContext.Provider
diff --git a/site/src/pages/WorkspacesPage/WorkspacesTable.tsx b/site/src/pages/WorkspacesPage/WorkspacesTable.tsx
@@ -530,62 +530,64 @@ const WorkspaceActionsCell: FC<WorkspaceActionsCellProps> = ({
 
 	return (
 		<TableCell>
-			{abilities.actions.includes("start") && (
-				<PrimaryAction
-					onClick={() => startWorkspaceMutation.mutate({})}
-					isLoading={startWorkspaceMutation.isLoading}
-					label="Start workspace"
-				>
-					<PlayIcon />
-				</PrimaryAction>
-			)}
-
-			{abilities.actions.includes("updateAndStart") && (
-				<>
+			<div className="flex gap-1">
+				{abilities.actions.includes("start") && (
+					<PrimaryAction
+						onClick={() => startWorkspaceMutation.mutate({})}
+						isLoading={startWorkspaceMutation.isLoading}
+						label="Start workspace"
+					>
+						<PlayIcon />
+					</PrimaryAction>
+				)}
+
+				{abilities.actions.includes("updateAndStart") && (
+					<>
+						<PrimaryAction
+							onClick={() => {
+								workspaceUpdate.update(false);
+							}}
+							isLoading={workspaceUpdate.isUpdating}
+							label="Update and start workspace"
+						>
+							<PlayIcon />
+						</PrimaryAction>
+						<WorkspaceUpdateDialogs {...workspaceUpdate.dialogs} />
+					</>
+				)}
+
+				{abilities.actions.includes("stop") && (
 					<PrimaryAction
 						onClick={() => {
-							workspaceUpdate.update(false);
+							stopWorkspaceMutation.mutate({});
 						}}
-						isLoading={workspaceUpdate.isUpdating}
-						label="Update and start workspace"
+						isLoading={stopWorkspaceMutation.isLoading}
+						label="Stop workspace"
 					>
-						<PlayIcon />
+						<SquareIcon />
+					</PrimaryAction>
+				)}
+
+				{abilities.canCancel && (
+					<PrimaryAction
+						onClick={cancelBuildMutation.mutate}
+						isLoading={cancelBuildMutation.isLoading}
+						label="Cancel build"
+					>
+						<BanIcon />
 					</PrimaryAction>
-					<WorkspaceUpdateDialogs {...workspaceUpdate.dialogs} />
-				</>
-			)}
-
-			{abilities.actions.includes("stop") && (
-				<PrimaryAction
-					onClick={() => {
-						stopWorkspaceMutation.mutate({});
-					}}
-					isLoading={stopWorkspaceMutation.isLoading}
-					label="Stop workspace"
-				>
-					<SquareIcon />
-				</PrimaryAction>
-			)}
-
-			{abilities.canCancel && (
-				<PrimaryAction
-					onClick={cancelBuildMutation.mutate}
-					isLoading={cancelBuildMutation.isLoading}
-					label="Cancel build"
-				>
-					<BanIcon />
-				</PrimaryAction>
-			)}
-
-			{abilities.actions.includes("retry") && (
-				<PrimaryAction
-					onClick={retry}
-					isLoading={isRetrying}
-					label="Retry build"
-				>
-					<RefreshCcwIcon />
-				</PrimaryAction>
-			)}
+				)}
+
+				{abilities.actions.includes("retry") && (
+					<PrimaryAction
+						onClick={retry}
+						isLoading={isRetrying}
+						label="Retry build"
+					>
+						<RefreshCcwIcon />
+					</PrimaryAction>
+				)}
+			</div>
 		</TableCell>
 	);
 };
