ENGCOM-5362: Changed logic so that _scrollToTopIfVisible is called only if element is in viewport. Previously it was called only when the element was outside it. #23390

sidolov · web-flow · commit 1c65a6325726 · 2019-06-27T10:34:17.000-05:00
diff --git a/lib/web/mage/collapsible.js b/lib/web/mage/collapsible.js
@@ -570,7 +570,7 @@ define([
          * @private
          */
         _scrollToTopIfVisible: function (elem) {
-            if (this._isElementOutOfViewport(elem)) {
+            if (!this._isElementOutOfViewport(elem)) {
                 elem.scrollIntoView();
             }
         },

Original file line number	Diff line number	Diff line change
`@@ -570,7 +570,7 @@ define([`
`570`	`570`	`* @private`
`571`	`571`	`*/`
`572`	`572`	`_scrollToTopIfVisible: function (elem) {`
`573`		`- if (this._isElementOutOfViewport(elem)) {`
	`573`	`+ if (!this._isElementOutOfViewport(elem)) {`
`574`	`574`	`elem.scrollIntoView();`
`575`	`575`	`}`
`576`	`576`	`},`