Bug 1255955 - Scroll element into view when not visible; r?automatedtester draft
authorAndreas Tolfsen <ato@mozilla.com>
Fri, 05 Aug 2016 10:53:06 +0100
changeset 400450 72f96200c7ee05aea61b4e5d29ab3ac9a0e9be31
parent 400449 1761b5d4e0e6f4a758605c1ccc8f439f6cf8664d
child 400451 d0d27e6b2704ff5eef448c829ddd0430fea6e305
push id26147
push userbmo:ato@mozilla.com
push dateSat, 13 Aug 2016 19:00:38 +0000
reviewersautomatedtester
bugs1255955
milestone51.0a1
Bug 1255955 - Scroll element into view when not visible; r?automatedtester The if-condition in the specification compatible interactability check for interaction.clickElement is wrong. It should scroll an element into view when it is _not_ visible. If it is visible it does not matter. MozReview-Commit-ID: 2n34QddDkQv
testing/marionette/interaction.js
--- a/testing/marionette/interaction.js
+++ b/testing/marionette/interaction.js
@@ -87,17 +87,17 @@ this.interaction = {};
  *     Use WebDriver specification compatible interactability definition.
  */
 interaction.clickElement = function(el, strict = false, specCompat = false) {
   let win = getWindow(el);
 
   let visible = false;
   if (specCompat) {
     visible = element.isInteractable(el);
-    if (visible) {
+    if (!visible) {
       el.scrollIntoView(false);
     }
   } else {
     visible = element.isVisible(el);
   }
 
   if (!visible) {
     throw new ElementNotVisibleError("Element is not visible");