Skip to content

Fix root-absolute /assets/movies gif paths (unblocks Jekyll 4)#118

Merged
jonfroehlich merged 1 commit into
mainfrom
fix/movie-gif-paths
Jun 23, 2026
Merged

Fix root-absolute /assets/movies gif paths (unblocks Jekyll 4)#118
jonfroehlich merged 1 commit into
mainfrom
fix/movie-gif-paths

Conversation

@jonfroehlich

Copy link
Copy Markdown
Member

4 animation refs used root-absolute /assets/movies/… but the files live in module asset folders. The old jekyll-relative-links (github-pages gem) silently rescued them; the newer one under Jekyll 4 (#117) doesn't, so the html-proofer link-check gate flagged them. Page-relative paths resolve under both Jekyll 3.9.2 and 4.

Surfaced by the link-check gate validating the Jekyll 4 upgrade (#117). Part of #81/#110.

These 4 animation refs pointed at /assets/movies/ (root), but the files live in
module asset folders (sensors/assets/movies, arduino/assets/movies). The old
jekyll-relative-links bundled with the github-pages gem silently "rescued" them
by resolving to the page's module folder; the newer jekyll-relative-links under
Jekyll 4 (PR #117) does not, so html-proofer's link-check correctly flags them.

Make the refs page-relative (assets/movies/...), which resolves correctly under
both Jekyll 3.9.2 and 4. Surfaced by the html-proofer gate validating the Jekyll
4 upgrade. Part of #81 / #110.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jonfroehlich jonfroehlich added the content Lesson content: errata, corrections, new sections, pedagogy label Jun 23, 2026
@jonfroehlich jonfroehlich merged commit 92480ec into main Jun 23, 2026
3 checks passed
@jonfroehlich jonfroehlich deleted the fix/movie-gif-paths branch June 23, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Lesson content: errata, corrections, new sections, pedagogy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant