Improve card metadata display and styling#258
Merged
Merged
Conversation
Show /images/opensource-og-2.png for blog posts that don't have a thumbnail image, but only on list pages. The fallback is handled in JavaScript during client-side rendering of item cards. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Replace opensource-og-2.png with pos-og.png as the fallback image for blog posts without thumbnails on list pages. Remove unused blog-placeholder.svg and old OG image. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
gregswinehart
requested review from
cwickham,
ivelasq and
jeroenjanssens
as code owners
✅ Deploy Preview for posit-open-source canceled.
|
gregswinehart
changed the title
gregswinehart
changed the title
Blog YAML Checks✅ All 1 posts passed. |
cwickham
added a commit
that referenced
this pull request
May 21, 2026
PR #258 added images/outgrowing.jpg under content/blog/positron/... just before the move commit was rebased; git's rename detection didn't extend the parent-dir rename to the newly added file, so it landed outside the ported/ tree. Move it into place so the post's image reference resolves.
cwickham
added a commit
that referenced
this pull request
May 21, 2026
* Rewrite content-path blog cross-links to permalinks Replaces 476 internal links of the form /blog/<source>/<post>/ with their permalink form /blog/YYYY-MM-DD_<slug>/ across 231 files. Decouples links from folder structure so a future reorganization (e.g. moving ported posts to a subdirectory) doesn't break them. Source files (.qmd, .Rmd, .Rmarkdown) and their rendered .md/.markdown outputs get the same rewrite so re-rendering won't undo it. Both pre- and post-rewrite Hugo builds produce identical broken-link output (0 broken across 29,619 internal links / 1,012 HTML files). * Report link/file counts in check-blog-links.py A "0 broken" result was masking a silent Hugo build failure that produced no HTML at all. The script now reports how many internal links it actually checked across how many files, and exits non-zero with a stderr warning when 0 files are scanned — so a broken build can't masquerade as a clean check. * Move ported posts under content/blog/ported/ Relocates all ported source dirs (ai, education, great-tables, plotnine, pointblank, positron, quarto, rstudio, shiny, tidyverse) from content/blog/<source>/ to content/blog/ported/<source>/. URLs are unaffected — the blog permalink template uses date + slug/basename, not the section path. Also includes a few link cleanups exposed by the move: - Rewrites reference-style markdown links ([label]: URL form) and absolutizes `../`-style relative paths in rendered files where they cross out of the source's own section. These slipped through the previous content-path rewrite. - Replaces broken refs to /blog/ai/2023-04-05-deep-learning-scientific- computing-r-torch/ (a post that was never created) with the book's actual external page. Hugo's GetPage was silently masking these pre-move; the move exposes them. - Updates two slug references that hardcoded source paths (content/featured/blog.md, content/test-cards/_index.md). Verification: post-move build is clean, with 0 broken / 29,672 internal links across 1,014 HTML files, matching the pre-move baseline. * Fix quarto.org link in Quarto 1.7 release post Source had [quarto.org](/) (correct on quarto.org, where the post originally lived). After porting, this resolved to the opensource.posit.co homepage rather than quarto.org. Use the explicit external URL so it works in both contexts. * Update docs and skills to reflect ported/ move Refresh references in the authoring guide, porting notes, editing-ported notes, the blog CLAUDE.md JSON example, and the /new-post skill so they reflect the new content/blog/ported/<source>/ layout. Also tightens the link guidance in _porting-notes.md and _editing-ported-posts.md to recommend permalink-form URLs (/blog/YYYY-MM-DD_slug/) rather than content-path-style URLs that depend on folder structure. * Move outgrow-your-laptop image into ported/ tree PR #258 added images/outgrowing.jpg under content/blog/positron/... just before the move commit was rebased; git's rename detection didn't extend the parent-dir rename to the newly added file, so it landed outside the ported/ tree. Move it into place so the post's image reference resolves.
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.
Summary
Changes
Metadata display improvements:
Visual polish:
Blog post improvements:
Test plan
🤖 Generated with Claude Code