Skip to content

Commit 3b0cd5b

Browse files
fix(site): polish table alignment for workspace proxies (#24538)
Fixes alignment issues in the workspace proxies table. ## Changes - **Status column alignment**: Removed `text-right` from the Status column header and the `justify-end` flex wrapper from status cells. Status indicator dots now align consistently across rows regardless of text width ("Healthy" vs "Not reachable"). - **Error/warning text alignment**: Changed padding from `px-12` (48px) to `px-14` (56px) so error/warning messages align with proxy name text, which starts after cell padding (12px) + avatar (32px) + gap (12px) = 56px. - **colSpan fix**: Corrected `colSpan={4}` to `colSpan={3}` to match the actual number of table columns. Relates to DES-22000 > 🤖 Generated by Coder Agents
1 parent bf885cc commit 3b0cd5b

2 files changed

Lines changed: 7 additions & 9 deletions

File tree

site/src/pages/UserSettingsPage/WorkspaceProxyPage/WorkspaceProxyRow.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -69,13 +69,11 @@ export const ProxyRow: FC<ProxyRowProps> = ({ proxy, latency }) => {
6969
/>
7070
</TableCell>
7171

72-
<TableCell className="status">
73-
<div className="flex items-center justify-end">{statusBadge}</div>
74-
</TableCell>
72+
<TableCell className="status">{statusBadge}</TableCell>
7573

7674
<TableCell
7775
className={cn(
78-
"text-sm text-right",
76+
"text-sm",
7977
latency
8078
? getLatencyColor(latency.latencyMS)
8179
: "text-content-secondary",
@@ -87,7 +85,7 @@ export const ProxyRow: FC<ProxyRowProps> = ({ proxy, latency }) => {
8785
{shouldShowMessages && (
8886
<TableRow>
8987
<TableCell
90-
colSpan={4}
88+
colSpan={3}
9189
className="!p-0 border-b-0 divide-y divide-solid overflow-clip"
9290
>
9391
<ProxyMessagesRow
@@ -142,7 +140,7 @@ const ProxyMessagesList: FC<ProxyMessagesListProps> = ({
142140
}
143141

144142
return (
145-
<div className="bg-surface-primary px-12 py-4 border-0">
143+
<div className="bg-surface-primary px-14 py-4 border-0">
146144
<div className={cn("mb-1 text-xs font-semibold", titleClassName)}>
147145
{title}
148146
</div>

site/src/pages/UserSettingsPage/WorkspaceProxyPage/WorkspaceProxyView.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ export const WorkspaceProxyView: FC<WorkspaceProxyViewProps> = ({
5656
<Table>
5757
<TableHeader>
5858
<TableRow>
59-
<TableHead className="w-[70%]">Proxy</TableHead>
60-
<TableHead className="w-[10%] text-right">Status</TableHead>
61-
<TableHead className="w-[20%] text-right">Latency</TableHead>
59+
<TableHead className="w-[60%]">Proxy</TableHead>
60+
<TableHead className="w-[20%]">Status</TableHead>
61+
<TableHead className="w-[20%]">Latency</TableHead>
6262
</TableRow>
6363
</TableHeader>
6464
<TableBody>

0 commit comments

Comments
 (0)