Merge pull request #593 from bernhardoj/fix/29921-remove-style-html-t…

…o-text Remove style tag when converting html to text
Expensify · Oct 27, 2023 · 82bfcd1 · 82bfcd1
2 parents e35083e + cae8908
commit 82bfcd1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/__tests__/ExpensiMark-HTMLToText-test.js b/__tests__/ExpensiMark-HTMLToText-test.js
@@ -127,3 +127,8 @@ test('Test new line replacement on blockquote with heading inside', () => {
     testString = '<blockquote><h1>heading A</h1><h1>heading B</h1></blockquote>';
     expect(parser.htmlToText(testString)).toBe('heading A\n\nheading B');
 });
+
+test('Test remove style tag', () => {
+    const testString = '<div><svg><style>.default-avatar_20_svg__st1{fill:#008c59}</style></svg><p>a text</p></div>';
+    expect(parser.htmlToText(testString)).toBe('a text');
+});
diff --git a/lib/ExpensiMark.js b/lib/ExpensiMark.js
@@ -382,6 +382,11 @@ export default class ExpensiMark {
                 regex: /<\/(blockquote|h1|pre)>(.|\s)/gm,
                 replacement: '\n$2',
             },
+            {
+                name: 'removeStyle',
+                regex: /<style>.*?<\/style>/gi,
+                replacement: '',
+            },
             {
                 name: 'stripTag',
                 regex: /(<([^>]+)>)/gi,