From 5ea1f0d4eb81790a949d709fcf8ff8ca70a33f23 Mon Sep 17 00:00:00 2001
From: Shubham Sawant <sawantshubham571@gmail.com>
Date: Mon, 12 Mar 2018 15:19:37 +0530
Subject: [PATCH] Change bringInView() implementation to support unsupported
 browsers

---
 src/core/element.js | 35 +++++++++++++++++++++++------------
 1 file changed, 23 insertions(+), 12 deletions(-)

diff --git a/src/core/element.js b/src/core/element.js
index 4f964d1..4f6e545 100644
--- a/src/core/element.js
+++ b/src/core/element.js
@@ -71,6 +71,17 @@ export default class Element {
     );
   }
 
+  /**
+   * Manually scrolls to the position of element if `scrollIntoView` fails
+   */
+  scrollManually() {
+    const elementRect = this.node.getBoundingClientRect();
+    const absoluteElementTop = elementRect.top + this.window.pageYOffset;
+    const middle = absoluteElementTop - (this.window.innerHeight / 2);
+
+    this.window.scrollTo(0, middle);
+  }
+
   /**
    * Brings the element to middle of the view port if not in view
    */
@@ -79,20 +90,20 @@ export default class Element {
       return;
     }
 
-    // If browser supports scrollIntoView, use that otherwise center it manually
-    if (this.node.scrollIntoView) {
-      const scrollIntoViewOptions = this.options.scrollIntoViewOptions || {
+    // If browser does not support scrollIntoView
+    if (!this.node.scrollIntoView) {
+      this.scrollManually();
+      return;
+    }
+
+    try {
+      this.node.scrollIntoView(this.options.scrollIntoViewOptions || {
         behavior: 'smooth',
         block: 'center',
-      };
-
-      this.node.scrollIntoView(scrollIntoViewOptions);
-    } else {
-      const elementRect = this.node.getBoundingClientRect();
-      const absoluteElementTop = elementRect.top + this.window.pageYOffset;
-      const middle = absoluteElementTop - (this.window.innerHeight / 2);
-
-      this.window.scrollTo(0, middle);
+      });
+    } catch (e) {
+      // `block` option is not allowed in older versions of firefox, scroll manually
+      this.scrollManually();
     }
   }