Skip to content

Commit 7c6446b

Browse files
chore(mobile): e2e review skips behaviour confirmed as intended
The scroll-edge header blur, rows under the floating tab bar, Settings' own search and filter, unlabelled Settings groups, edge-clipped scrolling rows, and wrapping at the largest text size were reported on every run. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 91992bd commit 7c6446b

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

‎scripts/mobile-e2e/review.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@ export async function reviewPose(result: PoseResult, outputDirectory: string): P
4141
``,
4242
`Report only real defects a user would notice: text clipped, truncated mid-word where it should wrap, or overlapping other content; controls overlapping, cut off, or under the status bar, notch, home indicator, or keyboard; content hidden behind navigation chrome; colours wrong for the ${pose.appearance} appearance or unreadable contrast; large unexplained empty areas or misaligned layout; error banners or red error screens.`,
4343
`Ignore: the fixture content itself, the status bar clock, a development-build badge, and anything that is plainly a deliberate design choice. Prefer no finding to a speculative one.`,
44+
// Each of these has been confirmed as intended; the review kept reporting them.
45+
`These are intended, never report them: content fading or showing through the header as it scrolls under it (the iOS scroll-edge blur); list rows scrolling under the floating glass tab bar; the search and filter buttons on the Settings header (search searches settings, the filter picks which environments settings apply to); Settings groups without a section header; chips or rows cut off at the edge of a list that scrolls; labels and rows wrapping onto more lines at the largest accessibility text size.`,
4446
`Use the exact step names above. Severity: high = broken or unusable, medium = clearly wrong, low = polish.`,
4547
].join("\n");
4648
const { stdout } = await run(

0 commit comments

Comments
 (0)