Skip to content

Fix Experiment Details desktop placement - #1437

Merged
JacobStephens2 merged 2 commits into
mainfrom
issue-1435
Oct 27, 2026
Merged

JacobStephens2 merged 2 commits into
mainfrom
issue-1435

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

Keep Experiment Details beside the Trials grid and sticky while scrolling. Place main and Details together after the shared header/navigation, so the live-status line cannot take the Details column. Document desktop and phone placement.

header                  header
navigation              navigation
main: Trials            Details (sticky)
live refresh status
footer                  footer

Closes #1435

Evidence

  • Before: node --test --test-name-pattern='Experiment Details stay beside' console/tests/experiments.test.cjs failed: Details was at y=1002.7 in a 1400×900 viewport. The empty status line occupied column 2 beside main (y=155.2, height=817.5), forcing Details into the next row.
    After: the retained regression passes against the rebuilt binary. It opens Eval task and Trial Details, scrolls, forces a refresh HTTP 503, verifies the visible warning stays in the main column, and checks recovery preserves Details.
  • cargo build --locked: passed.
  • npm test --prefix console: 106 passed, 0 failed, including 390 px phone and Inbox Details coverage.
  • /usr/bin/time -p scripts/affected-tests main: passed, exit 0. Site checks: 51 passed. Selected cargo nextest run: 3,139 passed, 69 skipped; 1,830.709 s test execution. Total affected-check wall time: 2,158.63 s, including site checks and Rust test compilation (3m 20s).
  • GitHub's Linux test job, Console/site browser jobs, build checks, and macOS gate passed on 315fe40a.

Seams under test: the served Console Experiment page in Chromium, reader clicks and scrolling, and live-refresh HTTP failure/recovery. Existing browser coverage exercises phone layouts and Inbox Details.

Independent review against main (41e0175b): Standards 0 findings; Spec 0 findings. Both axes read all three changed files. Unread changed files: none. Generated-output exclusions: none.

Unaddressed findings

  • Standards: none.
  • Spec: none.
  • Security: none.

Security review outcome

No unaddressed introduced findings.

Merge Danger

Door: two-way

Revert the CSS rule and its browser regression to roll back.

Blast Radius: layout

The shared Console stylesheet controls Details placement; the complete Console browser suite covers neighboring pages and phone layouts.

Built with codex codex-cli 0.164.0 · gpt-6.1-sol · xhigh

@JacobStephens2
JacobStephens2 merged commit fd0bad8 into main Oct 27, 2026
18 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-1435 branch October 27, 2026 14:27
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.

Experiment page Details open below the page, leaving the right column blank

1 participant