fix: prevent double spaces in toPlainText when span follows non-span (

#93)
portabletext · Nov 2, 2024 · 43b963f · 43b963f
1 parent 203ae7c
commit 43b963f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/toPlainText.ts b/src/toPlainText.ts
@@ -3,7 +3,7 @@ import type {ArbitraryTypedObject, PortableTextBlock} from '@portabletext/types'
 import {isPortableTextBlock, isPortableTextSpan} from './asserters'
 
 const leadingSpace = /^\s/
-const trailingSpace = /^\s/
+const trailingSpace = /\s$/
 
 /**
  * Takes a Portable Text block (or an array of them) and returns the text value

diff --git a/test/toPlainText.test.ts b/test/toPlainText.test.ts
@@ -134,3 +134,16 @@ test('toPlainText: does not add leading whitespace on span-hugging non-span', ()
     }),
   ).toEqual('Now that is an image.')
 })
+
+test('toPlainText: does not add leading whitespace on span-hugging non-span (trailing)', () => {
+  expect(
+    toPlainText({
+      _type: 'block',
+      children: [
+        {_type: 'span', text: 'Now that is a '},
+        {_type: 'image', src: '/some/image.png'},
+        {_type: 'span', text: 'beautiful image.'},
+      ],
+    }),
+  ).toEqual('Now that is a beautiful image.')
+})