Fix "<ol> tag is dropped from an ordered list" #913 (#914)

* Add test case * Fix
mozilla · Oct 17, 2024 · be6b1e7 · be6b1e7
1 parent 6243980
commit be6b1e7
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 1 deletion.
diff --git a/Readability.js b/Readability.js
@@ -196,7 +196,7 @@ Readability.prototype = {
     "UL",
   ]),
 
-  ALTER_TO_DIV_EXCEPTIONS: ["DIV", "ARTICLE", "SECTION", "P"],
+  ALTER_TO_DIV_EXCEPTIONS: ["DIV", "ARTICLE", "SECTION", "P", "OL", "UL"],
 
   PRESENTATIONAL_ATTRIBUTES: [
     "align",

diff --git a/test/test-pages/ol/expected-metadata.json b/test/test-pages/ol/expected-metadata.json
@@ -0,0 +1,9 @@
+{
+  "title": "",
+  "byline": null,
+  "dir": null,
+  "excerpt": "AI hasn’t meaningfully changed anything in cybersecurity so far. Deep fake phishing is still rare, L",
+  "siteName": null,
+  "publishedTime": null,
+  "readerable": false
+}
diff --git a/test/test-pages/ol/expected.html b/test/test-pages/ol/expected.html
@@ -0,0 +1,7 @@
+<div id="readability-page-1" class="page">
+    <ol>
+        <li>
+            <p>AI hasn’t meaningfully changed anything in cybersecurity so far. Deep fake phishing is still rare, L</p>
+        </li>
+    </ol>
+</div>
diff --git a/test/test-pages/ol/source.html b/test/test-pages/ol/source.html
@@ -0,0 +1,7 @@
+<html>
+    <body>
+        <ol>
+            <li><p>AI hasn’t meaningfully changed anything in cybersecurity so far. Deep fake phishing is still rare, L</p></li>
+        </ol>
+    </body>
+</html>