@zipify/wysiwyg 1.0.0-dev.58 → 1.0.0-dev.59

package/example/tooltip/Tooltip.js

@@ -25,7 +25,7 @@ export class Tooltip {
     static get popperStyles() {
         return {
             willChange: 'transform',
-            zIndex: 10001
+            zIndex: 999999
         };
     }
 

package/lib/services/ContentNormalizer.js

@@ -1,6 +1,7 @@
 import { ContextWindow } from './ContextWidnow';
 
 export class ContentNormalizer {
+    static PARSER = new DOMParser();
     static BLOCK_STYLES = ['text-align', 'line-height'];
     static BLOCK_NODE_NAMES = ['P', 'H1', 'H2', 'H3', 'H4'];
 
@@ -20,11 +21,17 @@ export class ContentNormalizer {
     ];
 
     static normalize(content) {
-        return new ContentNormalizer(content).normalize();
+        const options = {
+            content,
+            parser: ContentNormalizer.PARSER
+        };
+
+        return new ContentNormalizer(options).normalize();
     }
 
-    constructor(content) {
+    constructor({ content, parser }) {
         this._content = content;
+        this._parser = parser;
         this._dom = null;
     }
 
@@ -36,7 +43,7 @@ export class ContentNormalizer {
     }
 
     _normalizeTextContent() {
-        this._dom = new DOMParser().parseFromString(this._content, 'text/html');
+        this._dom = this._parser.parseFromString(this._content, 'text/html');
 
         this._iterateNodes(this._removeEmptyNodes, this._isBlockNode);
         this._iterateNodes(this._normalizeListItems, (node) => node.tagName === 'LI');
@@ -59,7 +66,7 @@ export class ContentNormalizer {
     }
 
     _removeEmptyNodes(node) {
-        if (node.childNodes.length === 0) {
+        if (!node.innerHTML.replace(/\n/g, '').trim()) {
             node.remove();
         }
     }
@@ -74,6 +81,8 @@ export class ContentNormalizer {
             fragment.append(node);
         };
 
+        this._assignElementProperties(itemEl, itemEl.parentElement, ContentNormalizer.BLOCK_STYLES);
+
         for (const node of children) {
             if (node.tagName === 'P') {
                 append(node);

package/lib/services/__tests__/ContentNormalizer.test.js

@@ -79,6 +79,20 @@ describe('normalize text content', () => {
         expect(ContentNormalizer.normalize(input)).toBe(output);
     });
 
+    test('should ignore space only nodes', () => {
+        const input = '<p>lorem ipsum 1</p><p> </p><p>lorem ipsum 2</p>';
+        const output = '<p>lorem ipsum 1</p><p>lorem ipsum 2</p>';
+
+        expect(ContentNormalizer.normalize(input)).toBe(output);
+    });
+
+    test('should ignore newline chapters only nodes', () => {
+        const input = '<p>lorem ipsum 1</p><p>\n</p><p>lorem ipsum 2</p>';
+        const output = '<p>lorem ipsum 1</p><p>lorem ipsum 2</p>';
+
+        expect(ContentNormalizer.normalize(input)).toBe(output);
+    });
+
     test('should not ignore setting', () => {
         const input = '<p style="text-decoration-line: underline;">lorem ipsum</p>';
         const output = '<p><span style="text-decoration-line: underline;">lorem ipsum</span></p>';
@@ -106,4 +120,11 @@ describe('normalize text content', () => {
 
         expect(ContentNormalizer.normalize(input)).toBe(output);
     });
+
+    test('should assign block styles from list to paragraph', () => {
+        const input = '<ul style="line-height: 2;"><li>lorem ipsum</li></ul>';
+        const output = '<ul style="line-height: 2;"><li><p style="line-height: 2;">lorem ipsum</p></li></ul>';
+
+        expect(ContentNormalizer.normalize(input)).toBe(output);
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zipify/wysiwyg",
-  "version": "1.0.0-dev.58",
+  "version": "1.0.0-dev.59",
   "description": "Zipify modification of TipTap text editor",
   "main": "dist/wysiwyg.mjs",
   "repository": {