Bug 905763 - Fix named anchors in various pages so that the Sandstone theme header can be set to a fixed position

2 files changed, 21 insertions, 20 deletions

diff --git a/js/comments.js b/js/comments.js
index 88ba49364..82c93e561 100644
--- a/js/comments.js
+++ b/js/comments.js
@@ -178,23 +178,3 @@ function getText(element) {
     }
     return text;
 }
-
-/**
- * If the URL contains a comment ID like #c10, scroll down the page to show the
- * entire comment below the fixed global header.
- */
-function scroll_comment_into_view() {
-    if (location.hash.match(/^#c\d+$/)) {
-        var $header = document.querySelector('#header');
-        var $comment = document.querySelector(location.hash);
-
-        if ($comment) {
-            window.setTimeout(function() {
-                window.scrollTo(0, $comment.offsetTop - $header.offsetHeight - 20);
-            }, 100);
-        }
-    }
-}
-
-window.addEventListener('load', scroll_comment_into_view, { once: true });
-window.addEventListener('hashchange', scroll_comment_into_view);
diff --git a/js/global.js b/js/global.js
index b7b517631..3a4074d27 100644
--- a/js/global.js
+++ b/js/global.js
@@ -190,3 +190,24 @@ $().ready(function() {
         $('.bz_autocomplete').attr('autocomplete', 'off');
     });
 });
+
+/**
+ * If the URL contains a hash like #c10, scroll down the page to show the
+ * element below the fixed global header. This workaround is required for
+ * comments on show_bug.cgi, components on describecomponents.cgi, etc.
+ */
+const scroll_element_into_view = () => {
+    if (location.hash) {
+        const $header = document.querySelector('#header');
+        const $comment = document.querySelector(location.hash);
+
+        if ($comment) {
+            window.setTimeout(() => {
+                window.scrollTo(0, $comment.offsetTop - $header.offsetHeight - 20);
+            }, 100);
+        }
+    }
+}
+
+window.addEventListener('load', scroll_element_into_view, { once: true });
+window.addEventListener('hashchange', scroll_element_into_view);