Skip to content

Commit 77803d7

Browse files
committed
fix(org): hold empty states, row dividers and edge actions to the page's column
An empty state sat 8px inside it, a divider ran 8px past it, a text button's label past it on touch, and an account's menu 15px short.
1 parent a4df4a8 commit 77803d7

5 files changed

Lines changed: 15 additions & 3 deletions

File tree

‎apps/sim/platform/design-system/components/empty-state/empty-state.module.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
.empty {
1010
justify-content: center;
1111
min-height: calc(var(--spacing-56) * 2);
12-
padding: var(--spacing-16) var(--spacing-8);
12+
padding-block: var(--spacing-16);
1313
color: var(--ui-ink);
1414
}
1515

‎apps/sim/platform/design-system/components/settings-section/settings-section.module.css‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,15 @@
132132
gap: 0;
133133
}
134134

135+
/* The divider stops with the text, where the row's hover fill reaches past it. */
135136
.rowList[data-dividers] > * + * {
137+
position: relative;
138+
}
139+
140+
.rowList[data-dividers] > * + *::before {
141+
position: absolute;
142+
inset-block-start: 0;
143+
inset-inline: var(--ui-row-inset);
136144
border-block-start: var(--spacing-stroke-default) solid var(--ui-divider);
145+
content: "";
137146
}

‎apps/sim/platform/embedded/app/(shell)/integrations/_components/connected-credential/connected-credential.module.css‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
flex: none;
2323
align-items: center;
2424
gap: var(--ui-stack-gap);
25+
margin-inline-end: calc(-1 * var(--ui-action-overhang));
2526
}
2627
.link {
2728
color: inherit;

‎apps/sim/platform/embedded/app/(shell)/settings/_components/sections/usage/usage-section.module.css‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,8 @@
149149
background: var(--control-danger);
150150
}
151151
.edgeAction {
152-
margin-inline-end: calc(-1 * var(--ui-action-overhang));
152+
/* A text button's own padding: its label ends on the column's edge, at any pointer. */
153+
margin-inline-end: calc(-1 * var(--spacing-8));
153154
}
154155
.tableHeading {
155156
display: flex;

‎apps/sim/platform/embedded/app/(shell)/settings/_components/settings-screen/settings-screen.module.css‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@
3131
}
3232
.docs {
3333
flex: none;
34-
margin-inline-end: calc(-1 * var(--ui-action-overhang));
34+
/* A text button's own padding: its label ends on the column's edge, at any pointer. */
35+
margin-inline-end: calc(-1 * var(--spacing-8));
3536
}
3637
.breadcrumb {
3738
display: flex;

0 commit comments

Comments
 (0)