content, client: additional-names-box: integrate more neatly

Since we're checking for [inert] ancestor anyway, we don't need
to depend on BS "first matched by querySelector()" logic anymore

2 files changed, 8 insertions, 6 deletions

diff --git a/src/content/dependencies/generateStickyHeadingContainer.js b/src/content/dependencies/generateStickyHeadingContainer.js
index 64becc5e..ec3062a3 100644
--- a/src/content/dependencies/generateStickyHeadingContainer.js
+++ b/src/content/dependencies/generateStickyHeadingContainer.js
@@ -33,12 +33,12 @@ export default {
           [
             html.tag('div', {class: 'content-sticky-heading-row'}, [
               html.tag('h1', [
-                slots.title,
-
-                // Placement after generally keeps the contents from being
-                // the first, when matched by .querySelector() calls.
                 html.tag('span', {class: 'reference-collapsed-heading'},
+                  {inert: true},
+
                   slots.title.clone()),
+
+                slots.title,
               ]),
 
               html.tag('div', {class: 'content-sticky-heading-cover-container'},
diff --git a/src/static/js/client/additional-names-box.js b/src/static/js/client/additional-names-box.js
index da1467ec..38e6d52f 100644
--- a/src/static/js/client/additional-names-box.js
+++ b/src/static/js/client/additional-names-box.js
@@ -30,8 +30,10 @@ export function getPageReferences() {
 
   info.stickyHeadingLink =
     document.querySelector(
-      '.content-sticky-heading-container ' +
-      'a[href="#additional-names-box"]');
+      '.content-sticky-heading-container' +
+      ' ' +
+      'a[href="#additional-names-box"]' +
+      ':not(:matches([inert] *))');
 
   info.contentContainer =
     document.querySelector('#content');