fmartingr/fmartingr.com
fmartingr/fmartingr.com
Watch
1
0
Fork
You've already forked fmartingr.com
 0
Code Issues 2 Pull requests 1 Projects Releases Packages Wiki Activity Actions

FMG-10: correct the STL previews guide and point it at forgejo-stlview #5

Open
butterrobot wants to merge 3 commits from butterrobot/fmg-10-stl-guide-corrections into master AGit
pull from: butterrobot/fmg-10-stl-guide-corrections
merge into: fmartingr:master
Conversation 0 Commits 3 Files changed 2 +46 -13
butterrobot commented 2026-09-10 16:00:05 +02:00
Contributor
Copy link

Corrects the STL previews post against the actual deployed setup. Full findings in FMG-10. Replaces the copy-Codeberg's-file instruction with a pointer to forgejo-stlview, and documents the volume mounts, the load-bearing classList.remove('stl-page'), and the three.js bare-specifier patch. Verified with hugo --minify (extended 0.165.0, matching CI).

Corrects the STL previews post against the actual deployed setup. Full findings in FMG-10. Replaces the copy-Codeberg's-file instruction with a pointer to forgejo-stlview, and documents the volume mounts, the load-bearing classList.remove('stl-page'), and the three.js bare-specifier patch. Verified with hugo --minify (extended 0.165.0, matching CI).
FMG-10: correct the STL previews guide and point it at forgejo-stlview
All checks were successful
Build and deploy / build-and-deploy (pull_request) Successful in 36s
Details
c1cec2ab53 
The guide told readers to copy Codeberg's stlview.js verbatim, but the
deployed viewer is a rewrite of it: it keys off the URL instead of the
"view raw" block, takes over .file-view, and loads three.js as ES modules
rather than through the UMD global. Following the guide as written left
you with a viewer that never loaded. Point at forgejo-stlview instead.

Also documents three things that were load-bearing but unstated: the
volume mounts (and that the container path is /data/gitea), the
classList.remove('stl-page') without which the step 2 CSS hides the
injected canvas, and the patch the three.js example modules need because
they import via the bare 'three' specifier.

Adds the file-view-only limitation, the [stlview] console-logging debug
hint, AGPL-3.0 attribution for the derived viewer, and the Forgejo
version this was verified against.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
FMG-10: reword the STL guide in the blog's own voice
Some checks failed
Build and deploy / build-and-deploy (pull_request) Has been cancelled
Details
ae32ef81f2 
Drops every em dash, ellipsis and curly quote from the prose. The
surrounding corpus uses straight apostrophes throughout and almost no em
dashes: the two cleanest technical posts have none at all, so the density
the previous commit introduced read as machine-written.

Rewrites the affected passages with the constructions used elsewhere on
the blog (colons, "since", "so", split sentences, first person) and drops
phrasing that does not appear in the corpus, such as "load-bearing" and
"is not a drop-in copy".

The file tree keeps its box-drawing characters since it is tree(1) output
inside a code fence and predates these edits. The viewer snippet now uses
the … escape, matching the deployed stlview.js exactly.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
butterrobot force-pushed butterrobot/fmg-10-stl-guide-corrections from ae32ef81f2
Some checks failed
Build and deploy / build-and-deploy (pull_request) Has been cancelled
Details
to c23c75317e
All checks were successful
Build and deploy / build-and-deploy (pull_request) Successful in 46s
Details
2026-09-11 08:27:55 +02:00 Compare
FMG-10: replace the STL preview screenshot
All checks were successful
Build and deploy / build-and-deploy (pull_request) Successful in 35s
Details
83d1f26f90 
New capture in the dark theme, showing a labelled screw tray instead of
the bracket. Same filename and alt text, so the post body is unchanged.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
All checks were successful
Build and deploy / build-and-deploy (pull_request) Successful in 35s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin +refs/pull/5/head:butterrobot/fmg-10-stl-guide-corrections
git switch butterrobot/fmg-10-stl-guide-corrections

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff butterrobot/fmg-10-stl-guide-corrections
git switch butterrobot/fmg-10-stl-guide-corrections
git rebase master
git switch master
git merge --ff-only butterrobot/fmg-10-stl-guide-corrections
git switch butterrobot/fmg-10-stl-guide-corrections
git rebase master
git switch master
git merge --no-ff butterrobot/fmg-10-stl-guide-corrections
git switch master
git merge --squash butterrobot/fmg-10-stl-guide-corrections
git switch master
git merge --ff-only butterrobot/fmg-10-stl-guide-corrections
git switch master
git merge butterrobot/fmg-10-stl-guide-corrections
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
fmartingr/fmartingr.com!5
No description provided.