refactor: Remove deprecated normalizeWhitespace option (#614)

fb55 · Aug 24, 2021 · 018ddf9 · 018ddf9
1 parent 3b01ac7
commit 018ddf9
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 144 deletions.
diff --git a/readme.md b/readme.md
@@ -76,77 +76,6 @@ When the parser is used in a non-streaming fashion, `endIndex` is an integer
 indicating the position of the end of the node in the document.
 The default value is `false`.
 
-## Option: `normalizeWhitespace` _(deprecated)_
-
-Replace all whitespace with single spaces.
-The default value is `false`.
-
-**Note:** Enabling this might break your markup.
-
-For the following examples, this HTML will be used:
-
-```html
-<font> <br />this is the text <font></font></font>
-```
-
-### Example: `normalizeWhitespace: true`
-
-```javascript
-[
-    {
-        type: "tag",
-        name: "font",
-        children: [
-            {
-                data: " ",
-                type: "text",
-            },
-            {
-                type: "tag",
-                name: "br",
-            },
-            {
-                data: "this is the text ",
-                type: "text",
-            },
-            {
-                type: "tag",
-                name: "font",
-            },
-        ],
-    },
-];
-```
-
-### Example: `normalizeWhitespace: false`
-
-```javascript
-[
-    {
-        type: "tag",
-        name: "font",
-        children: [
-            {
-                data: "\n\t",
-                type: "text",
-            },
-            {
-                type: "tag",
-                name: "br",
-            },
-            {
-                data: "this is the text\n",
-                type: "text",
-            },
-            {
-                type: "tag",
-                name: "font",
-            },
-        ],
-    },
-];
-```
-
 ---
 
 License: BSD-2-Clause

diff --git a/src/__fixtures__/16-normalize_whitespace.json b/src/__fixtures__/16-normalize_whitespace.json
diff --git a/src/index.ts b/src/index.ts
@@ -12,8 +12,6 @@ import {
 
 export * from "./node";
 
-const reWhitespace = /\s+/g;
-
 export interface DomHandlerOptions {
     /**
      * Add a `startIndex` property to nodes.
@@ -33,16 +31,6 @@ export interface DomHandlerOptions {
      */
     withEndIndices?: boolean;
 
-    /**
-     * Replace all whitespace with single spaces.
-     *
-     * **Note:** Enabling this might break your markup.
-     *
-     * @default false
-     * @deprecated
-     */
-    normalizeWhitespace?: boolean;
-
     /**
      * Treat the markup as XML.
      *
@@ -53,7 +41,6 @@ export interface DomHandlerOptions {
 
 // Default options
 const defaultOpts: DomHandlerOptions = {
-    normalizeWhitespace: false,
     withStartIndices: false,
     withEndIndices: false,
 };
@@ -165,23 +152,11 @@ export class DomHandler {
     }
 
     public ontext(data: string): void {
-        const { normalizeWhitespace } = this.options;
         const { lastNode } = this;
 
         if (lastNode && lastNode.type === ElementType.Text) {
-            if (normalizeWhitespace) {
-                lastNode.data = (lastNode.data + data).replace(
-                    reWhitespace,
-                    " "
-                );
-            } else {
-                lastNode.data += data;
-            }
+            lastNode.data += data;
         } else {
-            if (normalizeWhitespace) {
-                data = data.replace(reWhitespace, " ");
-            }
-
             const node = new Text(data);
             this.addNode(node);
             this.lastNode = node;