Scroll into view before moving pointer to make Firefox behave the same as Chrome

[alexpott]

Tried to get Drupal core tests running on firefox see https://git.drupalcode.org/project/drupal/-/jobs/2206793 … some are passing but a load fail due to https://stackoverflow.com/questions/44777053/selenium-movetargetoutofboundsexception-with-firefox

The webdriver-classic-driver does https://github.com/minkphp/webdriver-classic-driver/blob/73ad0b6ce21cf697c15d8131c32656cad8b8b0ec/src/WebdriverClassicDriver.php#L1007

[alexpott]

If I run the test added on this issue without the fix it passes on chrome but fails on firefox.

It fails with the expected error.

1) Behat\Mink\Tests\Driver\Custom\LargePageClickTest::testLargePageClick
WebDriver\Exception\MoveTargetOutOfBounds: Move target (786, 19488) is out of bounds of viewport dimensions (1360, 888)

[AlexSkrypnyk]

@longwave @alexpott
The addition of $this->scrollElementIntoView($element); broke multiple projects that have fixed top and bottom bars and where handling scrolling into view by themselves. This change reset that scrolling position and there is no way to preserve that position.

Is it possible to either revert this change or make it an opt-in/opt-out.

I'm happy to submit a PR if you could please let me know about the desired technical direction.