add captureBeyondViewport flag for Puppeteer

Since v5.3.7 Puppeteer does not capture beyond viewport anymore, which could cause some issues for people relying on it. A new flag `captureBeyondViewport` for toggling that setting is introduced in engineOptions as discussed in #1344.
garris · Nov 12, 2021 · e3a80e2 · e3a80e2
1 parent 6a30f1f
commit e3a80e2
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/core/util/runPuppet.js b/core/util/runPuppet.js
@@ -385,13 +385,10 @@ async function captureScreenshot (page, browser, selector, selectorMap, config,
           }
 
           const type = config.puppeteerOffscreenCaptureFix ? page : el;
-          const params = config.puppeteerOffscreenCaptureFix
-            ? {
-                captureBeyondViewport: false,
-                path: path,
-                clip: box
-              }
-            : { captureBeyondViewport: false, path: path };
+          const params = { captureBeyondViewport: config.captureBeyondViewport ?? false, path: path };
+          if (config.puppeteerOffscreenCaptureFix) {
+            params.clip = box;
+          }
 
           await type.screenshot(params);
         } else {