Repository navigation
Enlarge the boundary diagram, drop the date, restore full-width text - #6
Merged
Merged
Conversation
The diagram was the densest thing on the page and the hardest to read. Lanes widen from 380 to 410 units and boxes from 150 to 175 so the labels can go from 13px to 15px without "baseline + reform" overflowing its box; lane labels and crossing captions scale with them. The boundary line is now three segments, so "tool call" and "typed result" sit in gaps rather than having the dashed line run through them — centring them beside the line instead put them on top of the "Choose a tool" box. Verified by rendering the SVG and looking at it. Removed the publication date, and reverted the 78ch measure cap so running text and the subtitle use the full line again, both at the author's request. Line length was a review finding, so it is worth revisiting deliberately rather than by default. Mobile: the diagram keeps a 700px floor inside a container that scrolls, its padding no longer eats most of a phone screen, and the card-tabs header stacks instead of crowding. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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.
Three changes, all requested.
The boundary diagram is bigger. It is the densest thing on the page and was the hardest to read. Simply raising the type size overflowed the boxes — "baseline + reform" needs 153 units at 15px against a 150-unit box — so the lanes widen from 380 to 410 units and the boxes from 150 to 175, and the labels then go from 13px to 15px, with lane labels and crossing captions scaled to match.
The boundary line is now three segments rather than one, so
tool callandtyped resultsit in gaps in it. I tried centring them beside the line instead; that puttool callon top of the "Choose a tool and arguments" box. Verified by rendering the SVG and looking at it rather than reasoning about coordinates.Removed the publication date from the page and the Markdown copy.
Reverted the 78ch measure cap, so running text and the subtitle use the full line again. Worth noting that line length was a review finding — text ran to about 140 characters a line, against a typographic norm of 45–75 — so this is a deliberate choice to accept that rather than an oversight. Happy to revisit with a wider cap (say 90ch) if the middle ground appeals.
Mobile. The diagram keeps a 700px floor inside a container that scrolls, so its labels stay legible instead of shrinking to about 3.5px; its padding no longer eats most of a phone screen; and the card-tabs header stacks rather than crowding. Audited the rest of the breakpoints while I was in there — wrapper padding, tool cards and timeline collapsing to one column, the scrollytelling panel unstacking, and the results table scrolling all behave.
bun run checkpasses.