Repository navigation
Fix Experiment Details desktop placement - #1437
Merged
Merged
Conversation
This was referenced Oct 27, 2026
Closed
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Keep Experiment Details beside the Trials grid and sticky while scrolling. Place
mainand Details together after the shared header/navigation, so the live-status line cannot take the Details column. Document desktop and phone placement.Closes #1435
Evidence
node --test --test-name-pattern='Experiment Details stay beside' console/tests/experiments.test.cjsfailed: Details was at y=1002.7 in a 1400×900 viewport. The empty status line occupied column 2 besidemain(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. Selectedcargo 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).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
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