Skip to content

Commit 7503418

Browse files
committed
fix(org): let Home's workspace rows fill their section again
Adopting RowList dropped the rule that stretched the list past the viewport's justify-items: start. Each row's container containment gives it no intrinsic width, so the list shrank to a chevron, hiding every workspace's name and last run. Only the See more chip stays start-aligned.
1 parent d8c7055 commit 7503418

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

‎apps/sim/platform/app/(shell)/home/_components/home-workspaces/home-workspaces.module.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
display: grid;
33
grid-template-columns: minmax(0, 1fr);
44
gap: var(--ui-list-gap);
5-
justify-items: start;
65
}
76
/* Stays on the rows' outer edge, where the list's inset puts them. */
87
.viewport > :not(ul) {
8+
justify-self: start;
99
margin-inline-start: calc(-1 * var(--ui-row-inset));
1010
}

‎apps/sim/scripts/org-redesign-e2e/checks/home.ts‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -466,6 +466,13 @@ export async function homeChecks() {
466466
await expect(region.getByText('Last run failed: Invoice sync')).toBeVisible()
467467
await expect(region.getByText('2hrs ago')).toBeVisible()
468468
await expect(region.getByText('No recent workflow runs')).toBeVisible()
469+
const rowWidth = (await region.getByRole('link', { name: /Invoice sync/ }).boundingBox())
470+
?.width
471+
const regionWidth = (await region.boundingBox())?.width
472+
assert(
473+
rowWidth && regionWidth && rowWidth >= regionWidth,
474+
`A row is ${rowWidth}px wide in a ${regionWidth}px section, too narrow to show its name`
475+
)
469476
})
470477
})
471478
await check(

0 commit comments

Comments
 (0)