MDL-47353 gradereport_grader: Fix header offset logic for floating he…

…aders This is necessary for themes with non-fixed navbars to work with the grader report, otherwise headers will be hidden. This issue is a part of the MDL-46658 Task. This issue is a part of the MDL-25544 Epic.
patkira · Oct 3, 2014 · 3d8ef7c · 3d8ef7c
1 parent 6821f73
commit 3d8ef7c
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 3 deletions.
diff --git a/...e-gradereport_grader-gradereporttable/moodle-gradereport_grader-gradereporttable-debug.js b/...e-gradereport_grader-gradereporttable/moodle-gradereport_grader-gradereporttable-debug.js
@@ -485,6 +485,13 @@ FloatingHeaders.prototype = {
         if (header) {
             if (header.getComputedStyle('position') === 'fixed') {
                 this.pageHeaderHeight = header.get(OFFSETHEIGHT);
+            } else {
+                var navbar = Y.one('.navbar');
+
+                if (navbar) {
+                    // If the navbar exists and isn't fixed, we need to offset the page header to accommodate for it.
+                    this.pageHeaderHeight = navbar.get(OFFSETHEIGHT);
+                }
             }
         }
     },