Skip to content

FE-1548: Rework the optimization and sweep drawer views - #9422

Draft
kube wants to merge 1 commit into
cf/fe-1547-isolate-the-contour-and-sweep-components-with-dedicatedfrom
cf/fe-1548-rework-the-optimization-and-sweep-drawer-views
Draft

FE-1548: Rework the optimization and sweep drawer views#9422
kube wants to merge 1 commit into
cf/fe-1547-isolate-the-contour-and-sweep-components-with-dedicatedfrom
cf/fe-1548-rework-the-optimization-and-sweep-drawer-views

Conversation

@kube

@kube kube commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

🌟 What is the purpose of this PR?

Reorganizes the experiment and optimization result drawers so the graphs come before the event lists: the summary, the navigator, and the surface hold still at the top of the drawer, and only the step/metric lists scroll, each in its own container. Stacked on #9421.

🔗 Related links

🔍 What does this change?

  • The optimization drawer's section order is now Summary → Best parameters → Surface → Steps (the surface previously sat below the steps).
  • The experiment drawer's section order is now Summary → Parameters → Surface → Metrics (the surface previously sat below the metric charts).
  • Both drawer bodies are flex columns that no longer scroll as a whole: the sections above the list are fixed, and the steps table / metric grid scrolls in its own container filling the remaining height.
  • The steps table's header row stays pinned while the list scrolls (the sticky element is the header's rowgroup — a sticky row can only move within its own rowgroup, which is exactly as tall as the row).
  • Fixes the surface control row's clipped labels: the ds Select's root insists on min-width: min-content, which overflowed its 170px box over the neighbouring "Y"/"Metric" labels.
  • Updates the experiments and optimization user-guide pages for the new section order and scroll behaviour.

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • modifies an npm-publishable library and I have added a changeset file(s)

📜 Does this require a change to the docs?

The changes in this PR:

  • require changes to docs which are made as part of this PR

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

🛡 What tests cover this?

  • The existing petrinaut unit suite passes; the layout change itself is visual and was verified in Storybook (list scrolls in isolation, drawer body holds still, table header pinned — checked by DOM probes and screenshots).

❓ How to test this?

  1. Run yarn workspace @hashintel/petrinaut dev.
  2. Open "Simulate / SimulateView / Parameter sweep" and click the sweep row: the summary, sliders, and surface stay put; only the metric charts scroll.
  3. Open "Simulate / SimulateView / Run Supply Chain optimization", create a study over two parameters, and open its row: the surface sits above the steps, the steps list scrolls in place, and its header row stays pinned.

📹 Demo

(screenshots to be attached — files handed over alongside this PR)

🤖 Generated with Claude Code

@kube kube self-assigned this Aug 28, 2026
@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Ready Ready Preview Aug 29, 2026 2:52am
petrinaut Ready Ready Preview Aug 29, 2026 2:52am
petrinaut-docs Ready Ready Preview Aug 29, 2026 2:52am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
hashdotdesign-tokens Ignored Ignored Preview Aug 29, 2026 2:52am

Request Review

@github-actions github-actions Bot added area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team labels Aug 28, 2026
@kube
kube force-pushed the cf/fe-1548-rework-the-optimization-and-sweep-drawer-views branch from b83e049 to 1b11836 Compare August 28, 2026 22:39
@kube
kube force-pushed the cf/fe-1548-rework-the-optimization-and-sweep-drawer-views branch from 1b11836 to ca19a7b Compare August 28, 2026 22:56
@kube
kube force-pushed the cf/fe-1548-rework-the-optimization-and-sweep-drawer-views branch from ca19a7b to 4727638 Compare August 28, 2026 23:15
@kube
kube force-pushed the cf/fe-1548-rework-the-optimization-and-sweep-drawer-views branch from 4727638 to be2d422 Compare August 28, 2026 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

1 participant