fix: links (kolplattformen#64)

* fix: 🐛 Fixar links med mellanslag * fix: 🐛 Links med single quote
Zingo21 · Feb 12, 2021 · 905b893 · 905b893
1 parent 3edbf8c
commit 905b893
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/lib/parseHtml.test.ts b/lib/parseHtml.test.ts
@@ -17,9 +17,10 @@ describe('parseHtml', () => {
     it('handles links with spaces', () => {
       const html = `<div>
         <a href="/foo bar">Hello </a>
+        <a href='/foo bar'>Hello </a>
       </div>`
 
-      expect(trim(html)).toEqual('<div><a href="/foo%20bar">Hello</a></div>')
+      expect(trim(html)).toEqual('<div><a href="/foo%20bar">Hello</a><a href=\'/foo%20bar\'>Hello</a></div>')
     })
   })
   describe('toMarkdown', () => {

diff --git a/lib/parseHtml.ts b/lib/parseHtml.ts
@@ -10,17 +10,26 @@ export const trim = (html: string = ''): string => {
     .split('</')
     .map((token) => token.trim())
     .join('</')
+
+  let result = trimmed
   const rxSpaces = /href="(.*)"/g
-  const matches = trimmed.match(rxSpaces)
+  let matches = trimmed.match(rxSpaces)
+  if (matches) {
+    // eslint-disable-next-line no-restricted-syntax
+    for (const match of matches) {
+      result = result.replace(match, match.replace(/ /g, '%20'))
+    }
+  }
+
+  const rxSpacesSing = /href='(.*)'/g
+  matches = trimmed.match(rxSpacesSing)
   if (matches) {
-    let result = trimmed
     // eslint-disable-next-line no-restricted-syntax
     for (const match of matches) {
       result = result.replace(match, match.replace(/ /g, '%20'))
     }
-    return result
   }
-  return trimmed
+  return result
 }
 
 interface Node {