discourse/discourse
1
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-03-09 14:34:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

FIX: ensures the element is on the page before scroll with lockon (#8458)

Browse Source
This commit is contained in:
Joffrey JAFFEUX 2019-12-04 14:12:50 +01:00 committed by GitHub
parent be1510b09d
commit b74679f231
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 15 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
app/assets/javascripts/discourse/lib/lock-on.js.es6
View File

@ -35,7 +35,7 @@ export default class LockOn {
elementTop() {
const $selected = $(this.selector);
if ($selected && $selected.offset && $selected.offset()) {
if ($selected.length && $selected.offset && $selected.offset()) {
return $selected.offset().top - minimumOffset();
}
}
@ -49,22 +49,25 @@ export default class LockOn {
}
lock() {
let previousTop = this.elementTop();
const startedAt = new Date().getTime();
$(window).scrollTop(previousTop);
let previousTop = this.elementTop();
const interval = setInterval(() => {
const top = Math.max(0, this.elementTop());
const scrollTop = $(window).scrollTop();
if (!previousTop) {
previousTop = this.elementTop();
previousTop && $(window).scrollTop(previousTop);
} else {
const top = Math.max(0, this.elementTop());
const scrollTop = $(window).scrollTop();
if (typeof top === "undefined" || isNaN(top)) {
return this.clearLock(interval);
}
if (typeof top === "undefined" || isNaN(top)) {
return this.clearLock(interval);
}
if (!within(4, top, previousTop) || !within(4, scrollTop, top)) {
$(window).scrollTop(top);
previousTop = top;
if (!within(4, top, previousTop) || !within(4, scrollTop, top)) {
$(window).scrollTop(top);
previousTop = top;
}
}
// Commit suicide after a little while