Calculate WdElement visibleAt logic with scrollTop

ferpasri · ferpasri · commit a4ca5be0820f · 2026-07-09T14:36:28.000+02:00
diff --git a/webdriver/src/org/testar/monkey/alayer/webdriver/WdElement.java b/webdriver/src/org/testar/monkey/alayer/webdriver/WdElement.java
@@ -398,8 +398,8 @@ protected void fillScrollValues() {
 
   public boolean visibleAt(double x, double y) {
     int scrollLeft = (root == null) ? 0 : (int) root.scrollLeft;
-    int scrollHeight = (root == null) ? 0 : (int) root.scrollHeight;
-    return rect != null && rect.contains(x - scrollLeft, y - scrollHeight);
+    int scrollTop = (root == null) ? 0 : (int) root.scrollTop;
+    return rect != null && rect.contains(x - scrollLeft, y - scrollTop);
   }
 
   public boolean visibleAt(double x, double y, boolean obscuredByChildFeature) {

Original file line number	Diff line number	Diff line change
`@@ -398,8 +398,8 @@ protected void fillScrollValues() {`
`398`	`398`
`399`	`399`	`public boolean visibleAt(double x, double y) {`
`400`	`400`	`int scrollLeft = (root == null) ? 0 : (int) root.scrollLeft;`
`401`		`- int scrollHeight = (root == null) ? 0 : (int) root.scrollHeight;`
`402`		`- return rect != null && rect.contains(x - scrollLeft, y - scrollHeight);`
	`401`	`+ int scrollTop = (root == null) ? 0 : (int) root.scrollTop;`
	`402`	`+ return rect != null && rect.contains(x - scrollLeft, y - scrollTop);`
`403`	`403`	`}`
`404`	`404`
`405`	`405`	`public boolean visibleAt(double x, double y, boolean obscuredByChildFeature) {`