Skip to content

test(canvas): evaluate bounded page dividers - #44

Draft
markm39 wants to merge 3 commits into
fix/opennotes-testflightfrom
codex/opennotes-page-divider-audit
Draft

markm39 wants to merge 3 commits into
fix/opennotes-testflightfrom
codex/opennotes-page-divider-audit

Conversation

@markm39

@markm39 markm39 commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Evaluate Mobile Ink PR #66's bounded page dividers against dense OpenNotes notebooks. This experimental dependency build does not change the accepted TestFlight build 40.

The native interaction and repeated-zoom tests passed, but the responsiveness audit failed. One simulator run recorded no main-queue stalls in the pan/zoom interaction (maximum 91 ms), versus seven in the earlier baseline run, but its render p95 was 31.28 ms against a 16.7 ms budget; writing still recorded a 118 ms maximum and the pages/keyboard scenario reached 2,533 ms. These are single runs on hosted simulators and do not establish a causal improvement. A follow-up profile sampled substantial XCTest accessibility-query activity on the main thread, but does not isolate every measured delay.

The dependency archive was built from the referenced commit, and its package metadata, peer dependencies and integrity were verified. The next audit waits for the host to settle and fails if it cannot; it keeps the same responsiveness and frame-time limits. Keep this draft out of release until the required responsiveness audit and real-device zoom and page-jump verification pass.

A settled-host rerun also failed: pages/keyboard reached 2,493.53 ms, pan/zoom p95 35.8 ms and writing 249.61 ms. The corresponding baseline run recorded 1,230 ms, 19.38 ms and 100.13 ms respectively. These are single hosted-simulator measurements, so they do not establish a causal regression. Comparison audit. Keep the draft out of release.

This branch has not been deployed

No deployments
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