Skip to content

Serve documentation screenshots locally instead of hot-linking GitHub - #395

Merged
mlonsk merged 1 commit into
mainfrom
user-ml/localize-screenshots
Sep 15, 2026
Merged

mlonsk merged 1 commit into
mainfrom
user-ml/localize-screenshots

Conversation

@mlonsk

@mlonsk mlonsk commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Every image on the documentation site was loaded from a GitHub host at page render time, across four of them: raw.githubusercontent.com, user-images.githubusercontent.com, github.com and cloud.githubusercontent.com. That made the site's imagery a live dependency on a third party, and cloud.githubusercontent.com is a host GitHub retired years ago.

This PR does one thing: it brings those images into the repository and repoints the references at them.

What is in here

  • 89 image files added under content/assets/images/, 7.1 MB total.
  • 18 articles repointed at them. Only articles whose sole change is the image reference are included, so this PR has no prose edits to review.

Naming follows the convention established in "Updated screenshots (#382)": <page-slug>-NN.<ext>.

How the files were verified

Each download was checked by its magic bytes rather than by HTTP status, and its file extension taken from those bytes rather than from the URL. That is how a github.com/.../blob/... URL was caught serving an HTML page instead of an image, which had been rendering as a broken image on the live site.

Every image on the docs site was loaded from a GitHub host at page render
time: raw.githubusercontent.com, user-images.githubusercontent.com,
github.com and cloud.githubusercontent.com. That made the site's imagery a
live dependency on a third party, and cloud.githubusercontent.com is a host
GitHub retired years ago.

This commit downloads all 89 of those images into content/assets/images/ and
repoints the references at them. Each file was verified by its magic bytes
rather than by HTTP status, and its extension taken from the bytes, which is
how the one github.com/.../blob/ URL serving an HTML page instead of an image
was caught.

Two images that were not on GitHub are included for the same reason:
getting-started/editions.md used GitHub's newer extensionless attachment URL
format, and release-notes/3_14_0.md hot-linked docs.tabulareditor.com for an
image already present in this repository.

Only articles whose sole change is the image reference are in this commit.

Not fixed: the gif referenced by beta-18_1.md and beta-18_2.md returns 404
because the TabularEditor3/PublicPreview repository no longer exists, so it
has been broken on the live site for some time. It is left pointing at GitHub
rather than removed, since dropping it would delete content from a historical
release note.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-mud-081affe03-395.westeurope.azurestaticapps.net

@mlonsk
mlonsk requested a review from otykier September 15, 2026 18:39
@mlonsk
mlonsk merged commit 1d1c133 into main Sep 15, 2026
6 checks passed
@mlonsk
mlonsk deleted the user-ml/localize-screenshots branch September 15, 2026 18:44
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-mud-081affe03-395.westeurope.azurestaticapps.net

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.

2 participants