Reapply "Draging (which involves a click) also needs to scroll into v…

…iew" This reverts commit 761ea2e.
Lullabot · Jul 31, 2024 · fff9d26 · fff9d26
1 parent 761ea2e
commit fff9d26
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/Selenium2Driver.php b/src/Selenium2Driver.php
@@ -1023,16 +1023,17 @@ public function dragTo(string $sourceXpath, string $destinationXpath)
         $destination = $this->findElement($destinationXpath);
 
         if ($this->isW3C()) {
+            $this->scrollElementIntoView($source);
             $actions = array(
                 'actions' => [
                     [
                         'type' => 'pointer',
                         'id' => 'mouse1',
                         'parameters' => ['pointerType' => 'mouse'],
                         'actions' => [
-                            ['type' => 'pointerMove', 'duration' => 0, 'origin' => [Element::WEB_ELEMENT_ID => $this->findElement($sourceXpath)->getID()], 'x' => 0, 'y' => 0],
+                            ['type' => 'pointerMove', 'duration' => 0, 'origin' => [Element::WEB_ELEMENT_ID => $source->getID()], 'x' => 0, 'y' => 0],
                             ['type' => 'pointerDown', "button" => 0],
-                            ['type' => 'pointerMove', 'duration' => 0, 'origin' => [Element::WEB_ELEMENT_ID => $this->findElement($destinationXpath)->getID()], 'x' => 0, 'y' => 0],
+                            ['type' => 'pointerMove', 'duration' => 0, 'origin' => [Element::WEB_ELEMENT_ID => $destination->getID()], 'x' => 0, 'y' => 0],
                             ['type' => 'pointerUp', "button" => 0],
                         ],
                     ],