TEST

Author: timvandermeij

test/integration/reorganize_pages_spec.mjs

@@ -209,7 +209,7 @@ describe("Reorganize Pages View", () => {
       );
     });
 
-    it("should reorder thumbnails after dropping", async () => {
+    fit("should reorder thumbnails after dropping", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);
@@ -293,7 +293,7 @@ describe("Reorganize Pages View", () => {
       );
     });
 
-    it("should reorder thumbnails after dropping two non-adjacent pages", async () => {
+    fit("should reorder thumbnails after dropping two non-adjacent pages", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);
@@ -329,7 +329,7 @@ describe("Reorganize Pages View", () => {
       );
     });
 
-    it("should select the dropped page (bug 2010820)", async () => {
+    fit("should select the dropped page (bug 2010820)", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);
@@ -743,7 +743,7 @@ describe("Reorganize Pages View", () => {
       await closePages(pages);
     });
 
-    it("should check that a save is triggered", async () => {
+    fit("should check that a save is triggered", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);
@@ -2401,7 +2401,7 @@ describe("Reorganize Pages View", () => {
       await closePages(pages);
     });
 
-    it("should restore the post-move mapping when undoing a delete", async () => {
+    fit("should restore the post-move mapping when undoing a delete", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);
@@ -2643,7 +2643,7 @@ describe("Reorganize Pages View", () => {
       await closePages(pages);
     });
 
-    it("should hide the selection counter after dropping selected pages", async () => {
+    fit("should hide the selection counter after dropping selected pages", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
           await waitForThumbnailVisible(page, 1);

test/integration/test_utils.mjs

@@ -586,6 +586,7 @@ async function dragAndDrop(page, selector, translations, steps = 1) {
   }
   await page.mouse.up();
   await page.waitForSelector("#viewer:not(.noUserSelect)");
+  await page.waitForSelector("#thumbnailsView:not(.isDragging)");
 }
 
 function waitForPageChanging(page) {

web/pdf_thumbnail_viewer.js

@@ -1461,6 +1461,9 @@ class PDFThumbnailViewer {
             // First movement while dragging.
             this.#onStartDragging(thumbnail);
             const stopDragging = (_e, isDropping = false) => {
+              if (_e.target !== this.container) {
+                return;
+              }
               this.#onStopDragging(isDropping);
               pointerDownAC.abort();
             };