Skip to content

feat: run comparison - shared diff UI primitives - #2598

Open
camielvs wants to merge 1 commit into
cmp-02-graph-summary-logicfrom
cmp-03-diff-ui-primitives
Open

feat: run comparison - shared diff UI primitives#2598
camielvs wants to merge 1 commit into
cmp-02-graph-summary-logicfrom
cmp-03-diff-ui-primitives

Conversation

@camielvs

@camielvs camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Description

Third PR in the Compare Runs stack. Adds the small, shared UI building blocks that the later comparison views reuse, so they stay consistent. Nothing here is wired into a page yet.

  • CompareRunPicker — a list of recent runs used to pick a run for either side of the comparison.
  • DiffStatusBadge — the coloured Added / Removed / Changed / Unchanged badge.
  • RunTag / RunTags — the small "A" / "B" pills that mark which run a value belongs to.
  • FieldDiffRow — a single field's before/after values with expand-to-see-more for long values.
  • YamlDiffView — a side-by-side YAML diff of the two run specs (Monaco diff editor).

Related Issue and Pull requests

Part of the Compare Runs stack. Builds on #2597; the next PR (#2599) builds on this branch.

Type of Change

  • New feature

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

These are shared presentational components with no entry point of their own on this branch. They're exercised through the full feature.

Additional Comments

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

🎩 Preview

A preview build has been created at: cmp-03-diff-ui-primitives/0440072

camielvs commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 18500a7 to 4370669 Compare July 30, 2026 23:22
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from b799ace to dbfea62 Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 4370669 to cb9779a Compare July 31, 2026 00:24
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from dbfea62 to ec52e0b Compare August 14, 2026 18:17
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch 2 times, most recently from 45ac2fb to b881d15 Compare August 14, 2026 19:07
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch 2 times, most recently from f4272b7 to 0577e9a Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from b881d15 to 1f63058 Compare August 14, 2026 21:01
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 0577e9a to 5f5b90c Compare August 14, 2026 23:47
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 1f63058 to 8a70f0f Compare August 14, 2026 23:47
Comment thread src/routes/v2/pages/CompareView/components/RunTag.tsx Outdated
Comment thread src/routes/v2/pages/CompareView/components/DiffStatusBadge.tsx
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 8a70f0f to 46feccf Compare August 17, 2026 20:01
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 5f5b90c to 0fc0811 Compare August 17, 2026 20:01
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 46feccf to 907eae2 Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 0fc0811 to 5f39e5d Compare August 17, 2026 23:03
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from 5f39e5d to d28657f Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 907eae2 to 1a21fb6 Compare August 18, 2026 00:44
@camielvs
camielvs force-pushed the cmp-02-graph-summary-logic branch from d28657f to 10263af Compare August 18, 2026 00:56
@camielvs
camielvs force-pushed the cmp-03-diff-ui-primitives branch from 1a21fb6 to 0440072 Compare August 18, 2026 00:56
@camielvs
camielvs marked this pull request as ready for review August 18, 2026 01:20
@camielvs
camielvs requested a review from a team as a code owner August 18, 2026 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant