Improve jump-to-section scrolling

documize · Nov 8, 2019 · 60ef205 · 60ef205
1 parent 7ae8015
commit 60ef205
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/gui/app/components/document/view-content.js b/gui/app/components/document/view-content.js
@@ -45,12 +45,12 @@ export default Component.extend(Notifier, {
 	didInsertElement() {
 		this._super(...arguments);
 
-		this.jumpToSection(this.get('currentPageId'));
 	},
 
 	didRender() {
 		this._super(...arguments);
 
+		this.jumpToSection(this.get('currentPageId'));
 		this.contentLinkHandler();
 	},
 

diff --git a/gui/app/services/browser.js b/gui/app/services/browser.js
@@ -11,7 +11,6 @@
 
 import $ from 'jquery';
 import Service, { inject as service } from '@ember/service';
-import { schedule } from '@ember/runloop';
 
 export default Service.extend({
     sessionService: service('session'),
@@ -42,15 +41,17 @@ export default Service.extend({
     },
 
     scrollTo(id) {
-        schedule('afterRender', () => {
-            let elem = $(id).offset();
-            if (_.isUndefined(elem)) return;
+        let elem = $(id).offset();
+        if (_.isUndefined(elem)) return;
 
-            $('html, body').animate({
-                scrollTop: elem.top
-            }, 250);
-        });
-	},
+        $('html, body').animate({
+            scrollTop: elem.top
+        }, 250);
+    },
+
+    waitScrollTo(id) {
+        setTimeout(() => { this.scrollTo(id); }, 1000);
+    },
 
 	downloadFile(content, filename) {
 		let b = new Blob([content], { type: 'text/html' });