Ignore definitions in <del> markup (#1688)

close #1687
w3c · Oct 11, 2024 · edb269f · edb269f
1 parent 5b92955
commit edb269f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/browserlib/extract-dfns.mjs b/src/browserlib/extract-dfns.mjs
@@ -319,7 +319,8 @@ export default function (spec, idToHeading = {}) {
     })
     .filter(hasValidType)
     // Exclude IDL terms defined in a block that is flagged as to be excluded
-    .filter(node => !node.closest('.exclude'))
+    // or inside a <del>
+    .filter(node => !node.closest('.exclude,del'))
     // When the whole term links to an external spec, the definition is an
     // imported definition. Such definitions are not "real" definitions, let's
     // skip them.

diff --git a/tests/extract-dfns.js b/tests/extract-dfns.js
@@ -159,6 +159,10 @@ const tests = [
    html: "<dfn id=foo data-dfn-type=invalidtype>Foo</dfn>",
    changesToBaseDfn: []
   },
+  {title: "ignores dfns in a <del>",
+   html: "<del><dfn id=foo>Foo</dfn></del>",
+   changesToBaseDfn: []
+  },
   {title: "ignores dfns already exported",
    html: "<dfn id='foo' data-dfn-type='dfn' data-export>Foo</dfn>. <dfn id='foo2' data-export>Foo</dfn> is already exported.",
    changesToBaseDfn: [{ access: "public" }]