From e115bf67f70bf1466a7e968744bc29438ddf4b9d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Elton=20Gon=C3=A7alves=20Gomes?= <eltongonc@gmail.com>
Date: Mon, 3 Jul 2023 15:36:25 +0200
Subject: [PATCH] Fixed issue #145

---
 .../src/lib/guided-tour.component.ts          | 48 ++++++++-----------
 1 file changed, 19 insertions(+), 29 deletions(-)

diff --git a/projects/ngx-guided-tour/src/lib/guided-tour.component.ts b/projects/ngx-guided-tour/src/lib/guided-tour.component.ts
index 6359268..ff53bbb 100644
--- a/projects/ngx-guided-tour/src/lib/guided-tour.component.ts
+++ b/projects/ngx-guided-tour/src/lib/guided-tour.component.ts
@@ -184,43 +184,33 @@ export class GuidedTourComponent implements AfterViewInit, OnDestroy {
         this.updateStepLocation();
         // Allow things to render to scroll to the correct location
         setTimeout(() => {
-            if (!this.isOrbShowing && !this.isTourOnScreen()) {
-                if (this.selectedElementRect && this.isBottom()) {
+            if (!this.isOrbShowing && !this.isTourOnScreen() && this.selectedElementRect) {
+                let topPos;
+
+                if (this.isBottom()) {
                     // Scroll so the element is on the top of the screen.
-                    const topPos = ((this.windowRef.nativeWindow.scrollY + this.selectedElementRect.top) - this.topOfPageAdjustment)
+                    topPos = ((this.windowRef.nativeWindow.scrollY + this.selectedElementRect.top) - this.topOfPageAdjustment)
                         - (this.currentTourStep.scrollAdjustment ? this.currentTourStep.scrollAdjustment : 0)
                         + this.getStepScreenAdjustment();
-                    try {
-                        this.windowRef.nativeWindow.scrollTo({
-                            left: null,
-                            top: topPos,
-                            behavior: 'smooth'
-                        });
-                    } catch (err) {
-                        if (err instanceof TypeError) {
-                            this.windowRef.nativeWindow.scroll(0, topPos);
-                        } else {
-                            throw err;
-                        }
-                    }
                 } else {
                     // Scroll so the element is on the bottom of the screen.
-                    const topPos = (this.windowRef.nativeWindow.scrollY + this.selectedElementRect.top + this.selectedElementRect.height)
+                    topPos = (this.windowRef.nativeWindow.scrollY + this.selectedElementRect.top + this.selectedElementRect.height)
                         - this.windowRef.nativeWindow.innerHeight
                         + (this.currentTourStep.scrollAdjustment ? this.currentTourStep.scrollAdjustment : 0)
                         - this.getStepScreenAdjustment();
-                    try {
-                        this.windowRef.nativeWindow.scrollTo({
-                            left: null,
-                            top: topPos,
-                            behavior: 'smooth'
-                        });
-                    } catch (err) {
-                        if (err instanceof TypeError) {
-                            this.windowRef.nativeWindow.scroll(0, topPos);
-                        } else {
-                            throw err;
-                        }
+                }
+
+                try {
+                    this.windowRef.nativeWindow.scrollTo({
+                        left: null,
+                        top: topPos,
+                        behavior: 'smooth'
+                    });
+                } catch (err) {
+                    if (err instanceof TypeError) {
+                        this.windowRef.nativeWindow.scroll(0, topPos);
+                    } else {
+                        throw err;
                     }
                 }
             }