IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: include header offset relative to window (#11114)

Browse Source
This commit is contained in:
Kris 2020-11-04 21:14:17 -05:00 committed by GitHub
parent fa12302e77
commit 3ee216c036
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
app/assets/javascripts/discourse/app/lib/offset-calculator.js
View File

@ -7,7 +7,14 @@ export function minimumOffset() {
iPadNav = document.querySelector(".footer-nav-ipad .footer-nav"), iPadNav = document.querySelector(".footer-nav-ipad .footer-nav"),
iPadNavHeight = iPadNav ? iPadNav.offsetHeight : 0; iPadNavHeight = iPadNav ? iPadNav.offsetHeight : 0;
return header ? header.offsetHeight + iPadNavHeight : 0; // if the header has a positive offset from the top of the window, we need to include the offset
// this covers cases where a site has a custom header above d-header (covers fixed and unfixed)
const headerWrap = document.querySelector(".d-header-wrap"),
headerWrapOffset = headerWrap.getBoundingClientRect();
return header
? header.offsetHeight + headerWrapOffset.top + iPadNavHeight
: 0;
} }
export default function offsetCalculator() { export default function offsetCalculator() {