Wire up scrub + progressive animations (WebM) - #422
Merged
Conversation
Add a WebM animation path to the screenshot harness and two clips:
- lib/browser.mjs encodeWebm(): the Playwright-bundled ffmpeg is a stripped
--disable-everything build (decode mjpeg/vp8, mux webm only), so frames are
captured as JPEG and piped through image2pipe -> VP8 (no PNG decode, palette
filters, or gif muxer exist). Small, looping, muted output.
- capture.mjs: shots with output:"webm" + anim:{kind:"scrub"|"layers",...}
sweep the draw-range across frames (camera/theme fixed) with a final-frame hold.
- shots.manifest.json: scrub-sweep (segment scrub drawing the toolpath) and
layer-buildup (layer-range build-up = the progressive reveal), ~80-100 KB each.
- Embed as <video autoplay loop muted playsinline> (absolute URLs, so no TypeDoc
warning) in the feature gallery + the workers concept page.
- Coverage matrix + harness README updated.
Verified: valid VP8 720x536 ~3.5s; Pages build within budget; both webm copied
to docs-site/media; mid-frames confirm partial scrub / partial build-up.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to the public-product pass: the temporal animations flagged as the honest remainder.
What
output:"webm"+anim:{kind,frames,fps,holdFrames}sweeps a draw-range control across frames and encodes a small looping muted WebM. The Playwright-bundled ffmpeg is a stripped--disable-everythingbuild (decode mjpeg/vp8, mux webm only), so frames are captured as JPEG and piped viaimage2pipe→ VP8 — no PNG decode / palette / gif muxer exist.scrub-sweep.webm(segment scrub drawing the toolpath) andlayer-buildup.webm(layer-range build-up — the progressive reveal).<video autoplay loop muted playsinline>(absolute URLs → no TypeDoc warning) in the feature gallery + the workers concept page.Verified
docs-site/media; video tags present.Note
<video>plays on the Pages site; GitHub-README inline video is unreliable, so the README continues to link the gallery for motion. The true load-time progressive/holdreveal capture is still pending (the build-up clip stands in) — tracked in the matrix.🤖 Generated with Claude Code