webviews: a gallery entry for the Chief picture crop
#21737 added pages/home-channels/PictureCrop.tsx without a gallery entry, so test/gallery-coverage.test.ts fails on feat-cmux-next itself. On a PR, that finding fails only PRs that touch the file or the home-channels gallery: #21623 and #21747 so far.
The Chief sheet shows the crop only after a file is chosen, so the page entry never reaches it. This adds a standalone entry, pages.home-channels.picture-crop, using the page's real strings and a sample still drawn as an inline SVG. It has three variants: wide, tall, and zoomed and moved. I added an entry instead of an allowlist line because the allowlist is meant to only shrink.
Verification
- At fa5ed5c04, the feat-cmux-next tip,
bun test test/gallery-coverage.test.tsfails with+ "pages/home-channels/PictureCrop.tsx#PictureCrop". With this change: 6 pass. gallery-manifestandgallery-entry-storetests pass.bun run typecheckfinds no type errors, andvp fmt --checkis clean.- Not checked locally: how the entry renders. CI's gallery play renders every entry.
Changelog
none
🤖 Generated with Claude Code
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Adds a gallery entry for the Chief picture crop so gallery-coverage passes on feat-cmux-next. The component added in #21737 had no gallery entry, and since the crop only appears after a file is chosen, the page entry never covers it.
- Adds
pages.home-channels.picture-cropusing the page's real strings and an inline SVG sample so drag and zoom are visible. - Provides landscape, portrait, and zoomed-and-moved variants.
- Adds an entry instead of an allowlist line because the allowlist is meant to only shrink.