Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
Exact-head review and provenance for
Linux E2E run 38734128428 is active. Its API binds the pull-request event to head |
|
Actual Linux matrix from run 38734128428: 150 passed, two failed, preserving the failed pan assertions. Both the initial attempt and existing CI retry produced these same values:
All four cases at each width hit an SVG outside controls with Checkout logs bind this run to merge commit Inference: this experiment isolates a failing synthesized-gesture path while direct touch moves both the bare and production containers. It does not establish a product CSS defect or capture native DOM event streams. Independent interpretation is pending before any feature-test repair. This draft PR remains investigation-only and must never merge; #884 remains blocked until an independently reviewed repair passes its renewed gates. |
|
Draft check of
This comment is edited in place once the branch has been quiet for a couple of hours. The sweep never changes the PR state. |
Investigation only. Do not merge this PR into the feature branch or main.
PR #884 still fails its Linux mobile pan checks at 390px and 320px: the rebased feature head passed 150 tests and failed those two checks with zero scroll movement. This stacked diagnostic preserves the frozen experiment without changing product code or weakening assertions.
The matrix compares a minimal scroll container with the dense production graph, using Chromium's synthesized scroll gesture and frame-separated direct touch commands. It requires matching viewport and container geometry, an SVG hit outside controls, positive immediate pan movement and unchanged product labels. Matrix evidence is attached before the soft assertions, so failed controls or movement remain CI failures. It records command sequences and pan/geometry/hit/browser results, not the native DOM event stream.
Only two files differ from the feature base: the frozen browser diagnostic and one literal pull-request branch filter needed to run E2E on this stacked PR. The 64 changed lines and both file hashes match the preserved original diagnostic exactly. Two independent source/staged review passes verified that identity and the assertion/workflow contract. Prior owner-executed Mac fixture evidence passed four viewports and transfers through identical frontend sources and build inputs; Linux evidence remains pending in this PR.
Base:
feat/plan-draft-drag, not main. Related: #631, #884. This PR does not close either issue. Preserve the diagnostic branch and artifacts after investigation; any product fix requires separate review and renewed feature CI.