Merge pull request #2999 from magento-honey-badgers/MAGETWO-91504-pdp…

…-mobile [honey] MAGETWO-91504: Mobile PDP accordion widget hides accordion content on phones with iOS
magento · Aug 14, 2018 · 8c1bd4c · 8c1bd4c
2 parents 11ade1f + 4f6b8a2
commit 8c1bd4c
Showing 1 changed file with 22 additions and 0 deletions.
diff --git a/lib/web/mage/collapsible.js b/lib/web/mage/collapsible.js
@@ -448,6 +448,7 @@ define([
                 if (this.options.animate) {
                     this._animate(showProps);
                 } else {
+                    this._scrollToTopIfVisible(this.content.get(0).parentElement);
                     this.content.show();
                 }
                 this._open();
@@ -553,6 +554,27 @@ define([
                     }, 1);
                 });
             }
+        },
+
+        /**
+         * @param {HTMLElement} elem
+         * @private
+         */
+        _scrollToTopIfVisible: function (elem) {
+            if (this._isElementOutOfViewport(elem)) {
+                elem.scrollIntoView();
+            }
+        },
+
+        /**
+         * @param {HTMLElement} elem
+         * @private
+         * @return {Boolean}
+         */
+        _isElementOutOfViewport: function (elem) {
+            var rect = elem.getBoundingClientRect();
+
+            return rect.bottom < 0 || rect.right < 0 || rect.left > window.innerWidth || rect.top > window.innerHeight;
         }
     });