Give a card title its width back, and plant the pin in the corner - #114
Merged
Merged
Conversation
Merging the badge onto the title's line made the title a flex column between the badge and the marks: 76px of a 224px card, where a row of its own had given it 196. The head is a text block now, so the title flows past the badge and its second line runs the full width — 136px, measured. The pin leaves the line of text for the card's top-right corner, outside the band the copy and menu buttons occupy on hover. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
#71 merged the badge onto the title's line and bought the body two lines. It
charged the title for them, and that only became visible once the README was
refilmed: titles truncated three words in.
What went wrong, measured
As a flex row the title was a column between the badge and the marks. On a
224px card at 1920 wide:
Before #71 the title had a row of its own and wrapped inside 196px. Going to 76
is why every title ended in an ellipsis.
The head is a text block now —
-webkit-boxwith the clamp on it rather thanon the title — with the badge and the marks inline. The title flows past them, so
it starts after the badge and its second line runs from the left edge, full
width. The two-line budget #71 asked for is unchanged; it just belongs to the
head instead of the title.
The two paddings came down with measurements rather than guesses: the controls
start 6px past where the right reserve lands, and the tick moved to
left: 2pxso the gutter it needs is 8px instead of 14.
The pin leaves the sentence
whenever the title happened to end near the edge — and that second line cost the
body one of its own. "Welcome to DevBox" was doing exactly that.
It is now driven into the card's top-right corner, over the border and slightly
outside it. That spot is deliberate: the copy and ⋯ buttons sit inside at 8px
and 34px from the right edge, so anything drawn in that band would be covered the
moment the pointer arrived.
⚡and📎stay in the line — they say what the notecontains, where the pin says where the card is stuck.
Checked in all four states the ticket asked about (at rest, hovered, ticked,
ticked-while-hovered), on a card whose title wraps and on a todo list with no
badge. Nothing overlaps, and the pin clears the hover controls.
Checked locally
npm test968 passing,npm run lintclean,npm run test:e2e15/15 in 4:03.this replaces. It needs the pictures retaken once this lands.
🤖 Generated with Claude Code