Repository navigation
Conversation
|
No actionable comments were generated in the recent review. Walkthrough
Merge Risk:
|
miacycle
left a comment
There was a problem hiding this comment.
Nice catch, and thanks for the clear writeup tying it to #8344.
I verified the fix holds up: docker-kanvas-infoq.webp exists in the article directory, there is no .png by that name left, and darkthumbnail now matches thumbnail, so dark mode will stop pointing at a missing file.
One small check before this lands: could you confirm the dark-mode card for this post renders the thumbnail correctly in a local preview? The frontmatter change itself looks exactly right.
|
Thanks for the review @miacycle! Confirmed that the thumbnail asset resolves properly with |
Signed-off-by: Ayush <ayushdungarwal3445@gmail.com>
a97ac6e to
3c20db4
Compare
Signed-off-by: Ayush <ayushdungarwal3445@gmail.com>
|
Preview deployment for PR #8345 removed. This PR preview was automatically pruned because we keep only the 3 most recently updated previews on GitHub Pages to stay within deployment size limits. If needed, push a new commit to this PR to generate a fresh preview. |
miacycle
left a comment
There was a problem hiding this comment.
Thanks for confirming the dark-mode render. I re-checked the head commit: the one-line change matches issue #8344 exactly, docker-kanvas-infoq.webp exists in the article directory, there is no .png left, and darkthumbnail now matches thumbnail. Checks are green. Approving.
Description
This PR fixes #8344
In
src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx,darkthumbnailwas pointing to./docker-kanvas-infoq.png, which does not exist in the directory (the asset was converted to.webp). This updatesdarkthumbnailto point to./docker-kanvas-infoq.webpmatchingthumbnail.Notes for Reviewers
darkthumbnailfrom.pngto.webpinsrc/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx.Signed commits
Summary by CodeRabbit