Skip to content

Replace catalog previews with per-project SVG thumbnails - #752

Merged
kkd16 merged 1 commit into
mainfrom
catalog-svg-thumbnails
Aug 4, 2026
Merged

Replace catalog previews with per-project SVG thumbnails#752
kkd16 merged 1 commit into
mainfrom
catalog-svg-thumbnails

Conversation

@kkd16

@kkd16 kkd16 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace live catalog iframe previews with lazy-loaded static SVG images
  • require and validate a self-contained 16:10 public/thumbnail.svg in source and build output
  • add one stock Replace me with an SVG placeholder to the template and all 47 existing projects
  • update project guidance and catalog metadata generation

Verification

  • node --test scripts/validation.test.mjs
  • node scripts/discover-projects.mjs: 47 conforming, 0 rejected
  • node scripts/build-site.mjs: 47 published, 0 rejected after frozen install, lint, build, and output validation
  • browser smoke: 47 cards, 47 thumbnail images, 0 iframes
  • git diff --check

Merge note

This is intentionally a repo-wide catalog migration, so it touches the catalog infrastructure plus every project thumbnail. The single-project auto-merge policy will reject its scope; please merge it manually.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Out of scope. changes outside a project folder are never auto-merged (e.g. "AGENTS.md")

Change only one projects/<slug>/ folder, then push again. Run node scripts/verify-project.mjs <slug> first — it catches this locally.

@kkd16
kkd16 force-pushed the catalog-svg-thumbnails branch from c16c732 to 202e6fa Compare August 4, 2026 05:32

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Out of scope. changes outside a project folder are never auto-merged (e.g. "AGENTS.md")

Change only one projects/<slug>/ folder, then push again. Run node scripts/verify-project.mjs <slug> first — it catches this locally.

@kkd16
kkd16 merged commit b5b5706 into main Aug 4, 2026
1 of 2 checks passed
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