Repository navigation
Give the trial harness's gram the full page width - #338
Merged
Merged
Conversation
trial/index.html wrapped everything, the gram included, in the 1100px column its notes read comfortably in. A component that narrow is not what an analyst's page gives it: the control row collapses its guidance and squeezes the annotation tables below about 1180px, and the picture loses half the width a full-page gram has -- so the trial was showing a constrained layout as if it were the product. The gram mount now sits outside the column, spanning the page between the body's margins; the notes above and the footer below keep their column. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CrHtVussSR8ZxsxisaepJM
Contributor
PR PreviewPreview deployment is ready! View Preview Last updated: 2026-09-18T09:45:30.645Z |
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.
The trial harness was showing the component too narrow to be representative.
Why
trial/index.htmlwrapped everything, the gram included, in the 1100px column its notes read comfortably in. A component that narrow is not what an analyst's page gives it: the control row collapses its guidance and squeezes the annotation tables below about 1180px, and the picture loses half the width a full-page gram has, so the trial was showing a constrained layout as if it were the product.The change
The gram mount now sits outside the column, spanning the page between the body's margins; the notes above and the footer below keep their column. Markup only, with a comment in the page saying why the mount is where it is.
At a 1600px viewport the component is now 1552px wide against the 1100px column, and all five control-row columns show uncollapsed. The picture itself is still the player's fixed 900px render area until the expand toggle is used; that is the player's existing behaviour and is not changed here.
This commit was first pushed to #337's branch, but that PR had already merged, so it is re-based onto main here as its own PR.
Checks
yarn build:standalone && yarn build:site && yarn check:siteon this change: site check passes (10 curated pages reachable, no dead links)🤖 Generated with Claude Code
https://claude.ai/code/session_01CrHtVussSR8ZxsxisaepJM
Generated by Claude Code