Skip to content

docs: Replace PLACEHOLDER with local dive output image#945

Open
lcampanella98 wants to merge 1 commit into
buildpacks:mainfrom
lcampanella98:docs-build-outputs-placeholder
Open

docs: Replace PLACEHOLDER with local dive output image#945
lcampanella98 wants to merge 1 commit into
buildpacks:mainfrom
lcampanella98:docs-build-outputs-placeholder

Conversation

@lcampanella98

Copy link
Copy Markdown
Contributor

Fixes Issue

Fixes #785

Problem

The "Inspect your app" how-to page had a literal PLACEHOLDER where a screenshot of the dive multi-process-app output should be. Readers saw the word "PLACEHOLDER" instead of the image the surrounding text describes.

Solution

I replaced the placeholder with the dive output screenshot. Instead of hotlinking the image from imgur (the existing precedent in this repo), I committed the PNG into the theme's static assets and reference it locally:

  • Added themes/buildpacks/static/images/dive-inspect-app.png.
  • Referenced it as /images/dive-inspect-app.png, matching how every other diagram in the docs is linked.

This avoids a runtime dependency on imgur, which can prune images that go unviewed for a while. I also gave the image alt text, which the imgur references elsewhere lack.

Testing

  • Confirmed the committed PNG is the correct dive multi-process-app screenshot (shows Layers, Layer Details, Image Details, and Current Layer Contents).
  • Built the site locally and verified the image renders on the page in place of the old placeholder.

Additional Info

Reference the dive screenshot from static/images instead of imgur so
the image can't disappear if the imgur upload is pruned.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@lcampanella98 lcampanella98 requested a review from a team as a code owner June 11, 2026 17:11
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.

PLACEHOLDER needs to be populated with dive analysis

1 participant