Docs: strip 26 placeholder screenshots for 1.0 ship (captures tracked in #920)

[malpern]

Removes never-captured placeholder-*.png references from 7 guides so they don't render as broken images on the public site for 1.0. Prose + ASCII mockups stand on their own; real screenshots untouched. Exact capture list preserved in #920 for 1.1. Docs only.

[claude]

---

Code Review

Summary: Pure docs cleanup — removes 26 never-captured placeholder-*.png references from 7 guides so the public site ships without broken images. Prose and ASCII mockups stand on their own; real screenshots are untouched.

---

What this does

Strips all <!-- Screenshot: ... --> comment + image tag pairs from:

• guides/installation-wizard.md (4 images)
• guides/layers.md (2 images)
• guides/qmk-import.md (3 images)
• guides/remapping.md (4 images)
• guides/script-execution.md (3 images)
• guides/vallack-nav.md (7 images)
• guides/window-management.md (3 images)

Total: 26 removed — matches the PR title. ✅

---

Issues

One orphaned HTML comment in guides/vallack-nav.md

The image tag for placeholder-vallack-modifier-comparison.png is correctly removed, but the <!-- Diagram: Side-by-side comparison of standard CAGS vs Vallack modifier placement --> comment directly above it is left behind. HTML comments don't render to readers, so this won't cause a visible broken element — but it's a stale artifact that was only meaningful paired with the image. Worth removing in this pass since you're already touching the file.

---

Everything else

• All other removals are clean: each comment + image pair is fully excised together.
• Prose context around each removed image reads naturally without the placeholder — no awkward dangling references.
• No accidental prose deletions; surrounding content is intact in all 7 files.
• Zero additions, 76 deletions — exactly the right shape for this kind of cleanup.
• No test or code impact; docs-only change.

---

Verdict

Approve with one minor fix: remove the orphaned <!-- Diagram: ... --> comment from guides/vallack-nav.md. Everything else is correct and ready to ship.

[claude]

Code Review PR 921: Strip 26 placeholder screenshots for 1.0

Overall: Clean docs-only PR. The approach is correct -- prose and ASCII mockups stand on their own. One small inconsistency to fix before merge.

SUMMARY
Removes 26 placeholder-*.png image tags from 7 guide files, preventing broken images on the public site at 1.0 ship. No code changes. Capture list tracked in PR 920.

ISSUE: Orphaned HTML comment in guides/vallack-nav.md

Every other placeholder in this PR removes both the HTML comment AND the image tag. There is one exception: in vallack-nav.md, the comment about the CAGS vs Vallack modifier comparison diagram is left behind while its image tag is deleted. Per the PR goal of letting prose stand on its own, the comment should be removed too -- consistent with all other hunks. The 1.1 capture is already tracked in PR 920.

Fix: Remove the leftover HTML comment line in vallack-nav.md.

WHAT LOOKS GOOD

• All 26 image tags removed, matching the PR description count.
• Prose and ASCII mockups read cleanly with no comprehension gaps.
• PR description is clear: explains the why, links the tracking issue, scopes 1.0 vs 1.1.
• Zero risk to app behavior or tests.

SUGGESTIONS (non-blocking)

• After merge and publish, do a quick scan of the live site for any remaining placeholder- references in other guides.
• Confirm the gh-pages publish step runs after merge (Scripts/publish-guides.sh per CLAUDE.md).

VERDICT: Approve pending removal of the orphaned HTML comment in vallack-nav.md.