gui tests: plugin-move scrolls Zed's editor with the wheel and presses Move where it is — the test set .ebody's scrollTop to 120, which took the Move button out of sight, so Playwright's click scrolled the body back first (120 → 60 or 0, "the failure moved the editor", Chromium, failing in zh on 6e623d16 and in both languages after). It now wheels only as far as the button stays in sight, and presses it with the mouse, checking it is in sight; "the failure/the move moved the editor" are unchanged. Passes in Chromium and WebKit, en and zh, twice.

internal/gui/tests/plugin-move.test.cjs

@@ -116,12 +116,29 @@ for (const engine of (process.env.BROWSER ? [process.env.BROWSER] : ["chromium",
       assert.ok(!text.includes(pkg), "names the npm package in the sentence");
       assert.ok(text.includes(w.own), "doesn't say Zed stays signed in");
 
-      // the reader scrolled to the field: nothing the move does moves it
-      const body = ed.locator(".ebody");
-      const top = await body.evaluate((e) => { e.scrollTop = 120; return e.scrollTop; });
+      // the reader scrolled to the field: nothing the move does moves it.
+      // A wheel, as a reader's scroll (a script's scrollTop is put back), and
+      // only so far that the button stays in sight: a click on a button out
+      // of sight is Playwright's own scroll first, not the reader's
+      const body = ed.locator(".ebody"), box = await body.boundingBox();
+      const at = () => body.evaluate((e) => e.scrollTop);
+      await page.mouse.move(box.x + box.width / 2, box.y + box.height / 2);
+      for (let i = 0; i < 20 && (await btn.boundingBox()).y > box.y + 40; i++) {
+        const was = await at();
+        await page.mouse.wheel(0, 30);
+        await page.waitForTimeout(80);
+        if (await at() === was) break;
+      }
+      const top = await at();
       assert.ok(top > 0, "the editor doesn't scroll: the test proves nothing about keeping its place");
+      // a press where the button is, as the reader's mouse does
+      const press = async () => {
+        const b = await btn.boundingBox();
+        assert.ok(b.y >= box.y && b.y + b.height <= box.y + box.height, "the button is out of sight");
+        await page.mouse.click(b.x + b.width / 2, b.y + b.height / 2);
+      };
 
-      await btn.click();
+      await press();
       await page.waitForFunction((busy) => [...document.querySelectorAll("#modal .move button")].some((b) => b.textContent === busy), w.busy, { timeout: 300 });
       assert.equal(await btn.isDisabled(), true, "the button can be pressed again mid-move");
 
@@ -135,7 +152,7 @@ for (const engine of (process.env.BROWSER ? [process.env.BROWSER] : ["chromium",
       if (process.env.ARTIFACT_DIR) await ed.screenshot({ path: path.join(process.env.ARTIFACT_DIR, `plugin-move-failed-${engine}-${lang}.png`) });
 
       // tried again, it goes through: the same editor, turned over
-      await btn.click();
+      await press();
       const back = ed.locator(".move button", { hasText: w.back });
       await back.waitFor();
       assert.equal(await page.locator("#modal .editor").count(), 1, "the editor closed");