Skip to content

fix: stop long embed snippets blowing out page width#57

Merged
wavekat-eason merged 1 commit into
mainfrom
fix/star-tracker-embed-overflow
May 15, 2026
Merged

fix: stop long embed snippets blowing out page width#57
wavekat-eason merged 1 commit into
mainfrom
fix/star-tracker-embed-overflow

Conversation

@wavekat-eason
Copy link
Copy Markdown
Contributor

Summary

  • The embed dl used grid-template-columns: 110px 1fr — and 1fr is minmax(auto, 1fr), so the data column's min-content (a long Markdown URL) pushed the whole page past body { max-width: 760px }.
  • Switch to minmax(0, 1fr) so the column can shrink and the existing overflow-x: auto on the code/pre child can take over.

Test plan

  • Open /wavekat on a narrow viewport (and on dark mode) — embed rows should stay within the page; long snippets scroll horizontally inside their own row.

🤖 Generated with Claude Code

The embed grid used `1fr` for the data column, which expands to its
min-content size — a single long URL inside the Markdown snippet
pushed the page beyond the body's max-width on dark mode. Switch to
`minmax(0, 1fr)` so the column can shrink and the flex child's
overflow-x scroll takes over.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

@wavekat-eason wavekat-eason merged commit 7b6b096 into main May 15, 2026
4 checks passed
@wavekat-eason wavekat-eason deleted the fix/star-tracker-embed-overflow branch May 15, 2026 23:47
wavekat-eason pushed a commit that referenced this pull request May 15, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.0.21](wavekat-com-v0.0.20...wavekat-com-v0.0.21)
(2026-05-15)


### Features

* add star-tracker worker
([#52](#52))
([6f13ee1](6f13ee1))
* add wavekat-asr to ecosystem
([#50](#50))
([f26f29b](f26f29b))
* auto-theme star-tracker embed snippet
([#55](#55))
([4989603](4989603))
* star-tracker recent trend, range, per-repo charts
([#54](#54))
([7189f91](7189f91))


### Bug Fixes

* keep star-tracker count fresh on webhook
([#56](#56))
([1fdf264](1fdf264))
* stop long embed snippets blowing out page width
([#57](#57))
([7b6b096](7b6b096))
* wrap star-tracker header on mobile
([#53](#53))
([82e92b9](82e92b9))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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