Changed logic so that _scrollToTopIfVisible is called only if element is in viewport. Previously it was called only when the element was outside it. Also changed the logic of _isElementOutOfViewport to work correctly.

Author: oskarolaussen

lib/web/mage/collapsible.js

@@ -570,7 +570,7 @@ define([
          * @private
          */
         _scrollToTopIfVisible: function (elem) {
-            if (this._isElementOutOfViewport(elem)) {
+            if (!this._isElementOutOfViewport(elem)) {
                 elem.scrollIntoView();
             }
         },
@@ -583,7 +583,7 @@ define([
         _isElementOutOfViewport: function (elem) {
             var rect = elem.getBoundingClientRect();
 
-            return rect.bottom < 0 || rect.right < 0 || rect.left > window.innerWidth || rect.top > window.innerHeight;
+            return rect.bottom > window.innerHeight || rect.right < 0 || rect.left > window.innerWidth || rect.top < 0;
         }
     });