flexysnap 0.4.0 → 0.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flexysnap",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Flexible visual snapshot testing for Playwright, built for e-commerce.",
   "keywords": [
     "playwright",

package/src/wireframeUtils.js

@@ -145,6 +145,13 @@ async function extractElementGroupData(page, elementGroup, groupIndex, elementId
                     };
                 }
 
+                function isRectWithinViewport(rect) {
+                    return rect.right >= 0 &&
+                        rect.left < window.innerWidth &&
+                        rect.bottom >= 0 &&
+                        rect.top < window.innerHeight;
+                }
+
                 function isElementVisible(element) {
                     const style = window.getComputedStyle(element);
                     if (style.display === 'none' ||
@@ -155,12 +162,9 @@ async function extractElementGroupData(page, elementGroup, groupIndex, elementId
                         return false;
                     }
 
-                    const rect = element.getBoundingClientRect()
+                    const rect = element.getBoundingClientRect();
 
-                    return rect.right >= 0 &&
-                        rect.left < window.innerWidth &&
-                        rect.bottom >= 0 &&
-                        rect.top < window.innerHeight;
+                    return isRectWithinViewport(rect);
                 }
 
                 function getElementsWithDirectText(root, textIgnoreClasses) {
@@ -237,9 +241,14 @@ async function extractElementGroupData(page, elementGroup, groupIndex, elementId
                             if (isNested)
                                 continue;
 
+                            const boundingRect = getTextNodeBoundingBox(descendant);
+
+                            if (!isRectWithinViewport(boundingRect))
+                                continue;
+
                             texts.push({
                                 text: text,
-                                boundingRect: getTextNodeBoundingBox(descendant)
+                                boundingRect: boundingRect
                             });
                         }
                     }