Skip to content

fix(og-image): restore space between "for" and "agents" in tagline#235

Merged
raymondk merged 1 commit into
mainfrom
fix/og-image-tagline-spacing
Jun 30, 2026
Merged

fix(og-image): restore space between "for" and "agents" in tagline#235
raymondk merged 1 commit into
mainfrom
fix/og-image-tagline-spacing

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Reported from an X link preview: the social card read "A library foragents" — no space between "for" and "agents".

Cause

In public/og-image.svg the tagline is split across <tspan> elements. The trailing space inside the first tspan (A library for ) was collapsed at render time, so "for" and "agents" ran together in the rasterized og-image.png that X actually displays.

Fix

  • Replaced the collapsible trailing space with a geometric dx="9" offset on the agents tspan. A non-breaking space (&#160;) did not survive the serif fallback font during rsvg-convert, but dx renders reliably.
  • Regenerated public/og-image.png per the repo workflow (rsvg-convert -w 1200 -h 630 public/og-image.svg -o public/og-image.png).

The live homepage tagline was already correct — this only affected the social-card image.

Note

X caches OG images aggressively. After deploy, the card may need a re-scrape via the X Post Inspector / Card Validator (or wait for cache expiry) before the corrected image appears.

The trailing space inside the first <tspan> was collapsed at render time,
producing "foragents" in the social card shown on X. Replace it with a
geometric dx offset on the "agents" tspan (a non-breaking space did not
survive the serif fallback font) and regenerate og-image.png.
@marc0olo marc0olo requested review from a team and JoshDFN as code owners June 30, 2026 11:25
@github-actions

Copy link
Copy Markdown

Skill Validation Report

No skill files were changed in this PR — validation skipped.

@raymondk raymondk merged commit b3263cd into main Jun 30, 2026
6 checks passed
@raymondk raymondk deleted the fix/og-image-tagline-spacing branch June 30, 2026 12:51
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.

2 participants