FIX: make full height menu and cloak respect custom headers

FIX: make full height menu and cloak respect custom headers

diff --git a/app/assets/javascripts/discourse/components/site-header.js.es6 b/app/assets/javascripts/discourse/components/site-header.js.es6
index ab765a0..815d323 100644
--- a/app/assets/javascripts/discourse/components/site-header.js.es6
+++ b/app/assets/javascripts/discourse/components/site-header.js.es6
@@ -367,7 +367,7 @@ const SiteHeaderComponent = MountWidget.extend(Docking, PanEvents, {
           $headerCloak.show();
         }
 
-        const menuTop = this.site.mobileView ? 0 : headerHeight();
+        const menuTop = this.site.mobileView ? headerTop() : headerHeight();
 
         let height;
         const winHeightOffset = 16;
@@ -386,6 +386,7 @@ const SiteHeaderComponent = MountWidget.extend(Docking, PanEvents, {
         }
         if (style.top !== menuTop + "px" || style.height !== height) {
           $panel.css({ top: menuTop + "px", height });
+          $(".header-cloak").css({ top: menuTop + "px" });
         }
         $("body").removeClass("drop-down-mode");
       }
@@ -434,3 +435,9 @@ export function headerHeight() {
     $header.outerHeight() + headerOffsetTop - $(window).scrollTop()
   );
 }
+
+export function headerTop() {
+  const $header = $("header.d-header");
+  const headerOffset = $header.offset();
+  return headerOffset ? headerOffset.top : 0;
+}

GitHub
sha: a61d9deb

1 Like

This commit has been mentioned on Discourse Meta. There might be relevant details there:

Revert "FIX: make full height menu and cloak respect custom headers"