Skip to content

Commit 5920ead

Browse files
committed
fix: Align Compare status ticks when a row also shows only here
1 parent dce27da commit 5920ead

2 files changed

Lines changed: 11 additions & 3 deletions

File tree

‎packages/app/src/components/workbench/compare/markers.ts‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,13 @@ export function renderMarker(
9898
}
9999
const statusMarker = renderStatusMarker(cmd, step, allCmdsThisSide)
100100
if (kind === 'missing' && !oneSideEntirelyEmpty) {
101-
return html`${statusMarker}<span
101+
// `only here` before the status so the ✓ stays in the right-edge column,
102+
// aligned with rows that show only a ✓.
103+
return html`<span
102104
class="marker info"
103105
title="Only present on this side — the other run ended before this step"
104106
>only here</span
105-
>`
107+
>${statusMarker}`
106108
}
107109
return statusMarker
108110
}

‎packages/app/src/components/workbench/compare/styles.ts‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,13 +201,19 @@ export const compareStyles = css`
201201
}
202202
203203
/* ── Per-cell status markers ── */
204+
/* Only the first marker pushes the group to the right edge; any following
205+
marker (e.g. the ✓ after an "only here" tag) sits adjacent, so the ✓
206+
stays in the same right-edge column across rows. */
204207
.marker {
205-
margin-left: auto;
208+
margin-left: 8px;
206209
font-family: var(--vscode-font-family, sans-serif);
207210
font-size: 10px;
208211
padding: 1px 7px;
209212
border-radius: 5px;
210213
}
214+
.step-cell .marker:first-of-type {
215+
margin-left: auto;
216+
}
211217
.marker.ok {
212218
padding: 0;
213219
background: transparent;

0 commit comments

Comments
 (0)