Skip to content

fix: keep scroll time indicator background behind its label - #182

Merged
touch2be merged 1 commit into
akemmanuel:masterfrom
touch2be:fix/scroll-time-indicator-pill
Oct 26, 2026
Merged

touch2be merged 1 commit into
akemmanuel:masterfrom
touch2be:fix/scroll-time-indicator-pill

Conversation

@touch2be

Copy link
Copy Markdown
Collaborator

Summary

The floating date/time pill shown while scrolling the transcript (#164) rendered its background as a thin bar above the text instead of behind it.

The pill sits in an h-0 flex row (so it doesn't take layout space). Flex's default align-items: stretch stretched the span to that zero height, leaving only the padding and border with a background while the label overflowed below. Adding items-start lets the pill keep its natural height.

Test plan

  • Scroll a long conversation: the date/time label sits fully inside the rounded background

🤖 Generated with Claude Code

The indicator sits in an h-0 flex row, so the default align-items: stretch
collapsed the pill to zero height and the text overflowed below the
background. Align items to the start so the pill keeps its natural height.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@touch2be
touch2be merged commit 14ad82d into akemmanuel:master Oct 26, 2026
2 checks passed
@touch2be touch2be mentioned this pull request Oct 26, 2026
2 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant