Skip to content

ref(preprod): Update snapshot list framing#114448

Merged
mtopo27 merged 3 commits intomasterfrom
mtopo27/ref/snapshot-list-frame
May 1, 2026
Merged

ref(preprod): Update snapshot list framing#114448
mtopo27 merged 3 commits intomasterfrom
mtopo27/ref/snapshot-list-frame

Conversation

@mtopo27
Copy link
Copy Markdown
Contributor

@mtopo27 mtopo27 commented Apr 30, 2026

Refactor the preprod snapshots list so grouped and ungrouped variants share the same parent frame and align more closely with the target Figma layout. This keeps focused view behavior unchanged while setting up the shared framing needed for the follow-up focus-view refactor.

List Frame Layout

Snapshot groups now render through shared group and variant frame primitives. Group headers are contained rows with bottom borders, ungrouped snapshots still get the same parent container, and sibling variants are separated by full-width borders instead of old card gaps.

Split View Canvas

Split diff mode now wraps the base/head columns in a canvas with secondary background and primary border. Branch labels sit at the top of the canvas without their previous bottom border.

Move snapshot list items into shared group and variant frames so grouped and ungrouped variants follow the same layout. Add a split-view canvas and keep focused view behavior unchanged for the next PR.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@mtopo27 mtopo27 force-pushed the mtopo27/ref/snapshot-list-frame branch from a844ddb to fc4b9ca Compare April 30, 2026 20:06
@mtopo27 mtopo27 merged commit 18cfd5a into master May 1, 2026
71 checks passed
@mtopo27 mtopo27 deleted the mtopo27/ref/snapshot-list-frame branch May 1, 2026 00:41
mtopo27 added a commit that referenced this pull request May 1, 2026
Keep snapshot group context visible while scrolling long preprod
snapshot lists. This stacks on #114448 and adds a pinned group label
inside the virtualized list so reviewers can tell which grouped variants
are currently in view after the original group header scrolls away.

**Virtualized List Header**

The pinned label is driven by the active virtual row instead of native
sticky positioning inside transformed rows, which avoids moving every
mounted group header at once.

Refs #114448

---------

Co-authored-by: OpenAI Codex <noreply@openai.com>
cleptric pushed a commit that referenced this pull request May 5, 2026
Refactor the preprod snapshots list so grouped and ungrouped variants
share the same parent frame and align more closely with the target Figma
layout. This keeps focused view behavior unchanged while setting up the
shared framing needed for the follow-up focus-view refactor.

**List Frame Layout**

Snapshot groups now render through shared group and variant frame
primitives. Group headers are contained rows with bottom borders,
ungrouped snapshots still get the same parent container, and sibling
variants are separated by full-width borders instead of old card gaps.

**Split View Canvas**

Split diff mode now wraps the base/head columns in a canvas with
secondary background and primary border. Branch labels sit at the top of
the canvas without their previous bottom border.

---------

Co-authored-by: OpenAI Codex <noreply@openai.com>
Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
cleptric pushed a commit that referenced this pull request May 5, 2026
Keep snapshot group context visible while scrolling long preprod
snapshot lists. This stacks on #114448 and adds a pinned group label
inside the virtualized list so reviewers can tell which grouped variants
are currently in view after the original group header scrolls away.

**Virtualized List Header**

The pinned label is driven by the active virtual row instead of native
sticky positioning inside transformed rows, which avoids moving every
mounted group header at once.

Refs #114448

---------

Co-authored-by: OpenAI Codex <noreply@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants