Put the copy button on the title's row, and call it "Copy .md" - #74
Merged
Merged
Conversation
The button sat above the heading on a row of its own, which spent vertical
space on a control and read as a banner rather than as part of the title.
Floated rather than laid out in a flex row: the h1 it sits beside is rendered
markdown, so there is no wrapper to make a flex parent of. The title text flows
around it at every width — at 390px the longest title ("Opening and moving
between files") still has 209px on its first line and wraps cleanly below the
float, so this needs no breakpoint of its own. An earlier comment claimed there
was no room beside the h1 on a phone; measuring says otherwise.
"Copy .md" over "Copy markdown" — it names the thing the button hands you, and
takes the control from 141px to 88px, which is most of why the narrow case is
comfortable.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JGMQ49hvA936om3GQfNTAn
jnahian
added
the
web
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
1 participant
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.
Follow-up to #72. The button sat above the heading on a row of its own, spending vertical space on a control and reading as a banner rather than as part of the title.
Now floated, so it shares the title's first line. Floated rather than laid out in a flex row because the h1 it sits beside is rendered markdown — there is no wrapper to make a flex parent of.
Also renamed Copy markdown → Copy .md: it names the thing the button hands you, and takes the control from 141px to 88px.
On the mobile case
The original comment on this block claimed the h1 wraps on a phone and leaves no room beside it. That turned out to be wrong. Measured at 390px with the longest title on the site — "Opening and moving between files" — the first line still has 209px beside the float and the remaining lines wrap full-width below it. No breakpoint needed.
Verified
Driven in a browser at 1440px and 390px: button on the title's row at both, no collision with the heading text, label going
Copy .md→Copied→ back, and the click fetching/docs/navigating.md(200, 2,703 chars).🤖 Generated with Claude Code
https://claude.ai/code/session_01JGMQ49hvA936om3GQfNTAn