no-muse/musescore-downloader
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fix: btns position relative to the entire document instead of viewport

Browse source
This commit is contained in:
Xmader 2021-01-07 02:38:06 -05:00
parent d9d09c4e8f
commit 8250d80d4b
No known key found for this signature in database
GPG key ID: A20B97FB9EB730E4
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/btn.ts
View file

@ -113,9 +113,12 @@ export class BtnList {
} }
private _positionBtns (anchorDiv: HTMLDivElement, newParent: HTMLDivElement) { private _positionBtns (anchorDiv: HTMLDivElement, newParent: HTMLDivElement) {
const { top } = anchorDiv.getBoundingClientRect() let { top } = anchorDiv.getBoundingClientRect()
top += window.scrollY // relative to the entire document instead of viewport
if (top > 0) { if (top > 0) {
newParent.style.top = `${top}px` newParent.style.top = `${top}px`
} else {
newParent.style.top = '0px'
} }
} }
@ -140,7 +143,6 @@ export class BtnList {
const anchorDiv = this.getBtnParent() const anchorDiv = this.getBtnParent()
const pos = () => this._positionBtns(anchorDiv, newParent) const pos = () => this._positionBtns(anchorDiv, newParent)
pos() pos()
document.addEventListener('readystatechange', pos, { passive: true })
// reposition btns when window resizes // reposition btns when window resizes
window.addEventListener('resize', pos, { passive: true }) window.addEventListener('resize', pos, { passive: true })