Skip to content

Scenario 2: opt a post into coverage#4

Open
RisingOrange wants to merge 1 commit intovisual-difffrom
visual-diff-scenario-2
Open

Scenario 2: opt a post into coverage#4
RisingOrange wants to merge 1 commit intovisual-difffrom
visual-diff-scenario-2

Conversation

@RisingOrange
Copy link
Copy Markdown
Owner

Tests the scope comment's response to an annotation change (adds visualDiffEnabled: true to src/posts/australia.md).

@RisingOrange RisingOrange force-pushed the visual-diff-scenario-2 branch from 39ec202 to 4d84cdb Compare April 24, 2026 11:31
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 24, 2026

Visual diff coverage
commit 4fa089b · run #24897360091

Review snapshots in Chromatic

  • Pages: 15/18 covered. Included by default.
  • Posts: 7/103 covered. Excluded by default (posts share a layout).
Route breakdown
  • Pages excluded:
    • /quotes: ~28M-pixel desktop snapshot is over Chromatic's 25M cap
    • /submitted: post-form state
    • /verify: token-dependent
  • Posts covered:
    • /action
    • /australia
    • /funding
    • /if-anyone-builds-it-campaign
    • /join
    • /learn
    • /values
Not exercised
  • Pages that fetch from external APIs render against pinned fixtures, not live data.
  • Cross-origin iframes and API calls (Tally, Luma, Mapbox, analytics, newly added widgets) are aborted at the browser boundary. Containers render as empty. Check the Netlify preview deploy to verify their real rendering.
  • Only en locale is built.

tests/visual/README.md explains the annotation syntax and the coverage model.

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.

1 participant