Add expanded ellipses options

lib/index.js

@@ -38,8 +38,11 @@
  *   dash;
  *   when `'inverted'`, turns three dashes into an en dash and two into an em
  *   dash.
- * @property {boolean | null | undefined} [ellipses=true]
+ * @property {'unspaced' | 'spaced' | boolean | null | undefined} [ellipses=true]
  *   Transform triple dots (default: `true`), with or without spaces between.
+ *   when `true`, turns triple dots into ellipses (with or without spaces between);
+ *   when `'unspaced'`, turns triple dots into ellipses (only for those without spaces);
+ *   when `'spaced'`, turns triple dots into ellipses (only for those with spaces).
  * @property {QuoteCharacterMap | null | undefined} [openingQuotes]
  *   Opening quotes to use (default: `{double: '“', single: '‘'}`).
  * @property {boolean | null | undefined} [quotes=true]
@@ -91,7 +94,11 @@ export default function retextSmartypants(options) {
     methods.push(quotesDefault)
   }
 
-  if (settings.ellipses !== false) {
+  if (settings.ellipses === 'unspaced') {
+    methods.push(ellipsesUnspaced)
+  } else if (settings.ellipses === 'spaced') {
+    methods.push(ellipsesSpaced)
+  } else if (settings.ellipses !== false) {
     methods.push(ellipsesDefault)
   }
 
@@ -219,14 +226,30 @@ function dashesOldschool(_, node) {
  * @type {Method}
  */
 function ellipsesDefault(_, node, index, parent) {
-  const value = node.value
-  const siblings = parent.children
+  ellipsesUnspaced(_, node, index, parent)
+  ellipsesSpaced(_, node, index, parent)
+}
 
+/**
+ * Transform multiple dots into unicode ellipses (only for those without spaces).
+ *
+ * @type {Method}
+ */
+function ellipsesUnspaced(_, node) {
   // Simple node with three dots and without whitespace.
   if (/^\.{3,}$/.test(node.value)) {
     node.value = '…'
-    return
   }
+}
+
+/**
+ * Transform multiple dots into unicode ellipses (only for those with spaces).
+ *
+ * @type {Method}
+ */
+function ellipsesSpaced(_, node, index, parent) {
+  const value = node.value
+  const siblings = parent.children
 
   if (!/^\.+$/.test(value)) {
     return

test.js

@@ -554,3 +554,156 @@ test('Ellipses', async function (t) {
     }
   )
 })
+
+test('Ellipses (unspaced)', async function (t) {
+  const processor = retext().use(retextSmartypants, {ellipses: 'unspaced'})
+
+  await t.test('should replace three full stops', async function () {
+    assert.equal(
+      processor.processSync('Alfred... Bertrand.').toString(),
+      'Alfred\u2026 Bertrand.'
+    )
+  })
+
+  await t.test('should replace three initial full stops', async function () {
+    assert.equal(
+      processor.processSync('...Alfred Bertrand.').toString(),
+      '\u2026Alfred Bertrand.'
+    )
+  })
+
+  await t.test('should replace three final full stops', async function () {
+    assert.equal(
+      processor.processSync('Alfred Bertrand...').toString(),
+      'Alfred Bertrand\u2026'
+    )
+  })
+
+  await t.test('should replace three padded full stops', async function () {
+    assert.equal(
+      processor.processSync('Alfred ... Bertrand.').toString(),
+      'Alfred \u2026 Bertrand.'
+    )
+  })
+
+  await t.test(
+    'should replace three padded initial full stops',
+    async function () {
+      assert.equal(
+        processor.processSync('... Alfred Bertrand.').toString(),
+        '\u2026 Alfred Bertrand.'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three padded final full stops',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred Bertrand ...').toString(),
+        'Alfred Bertrand \u2026'
+      )
+    }
+  )
+
+  await t.test('should replace more than three full stops', async function () {
+    assert.equal(
+      processor.processSync('Alfred..... Bertrand.').toString(),
+      'Alfred\u2026 Bertrand.'
+    )
+
+    assert.equal(
+      processor.processSync('Alfred bertrand....').toString(),
+      'Alfred bertrand\u2026'
+    )
+
+    assert.equal(
+      processor.processSync('......Alfred bertrand.').toString(),
+      '\u2026Alfred bertrand.'
+    )
+  })
+
+  await t.test(
+    'should NOT replace less than three full stops',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred.. Bertrand.').toString(),
+        'Alfred.. Bertrand.'
+      )
+
+      assert.equal(
+        processor.processSync('Alfred bertrand. .').toString(),
+        'Alfred bertrand. .'
+      )
+
+      assert.equal(
+        processor.processSync('.Alfred bertrand.').toString(),
+        '.Alfred bertrand.'
+      )
+    }
+  )
+})
+
+test('Ellipses (spaced)', async function (t) {
+  const processor = retext().use(retextSmartypants, {ellipses: 'spaced'})
+
+  await t.test(
+    'should replace three padded full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred . . . Bertrand.').toString(),
+        'Alfred \u2026 Bertrand.'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three padded initial full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('. . . Alfred Bertrand.').toString(),
+        '\u2026 Alfred Bertrand.'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three padded final full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred Bertrand . . .').toString(),
+        'Alfred Bertrand \u2026'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred. . . Bertrand.').toString(),
+        'Alfred\u2026 Bertrand.'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three initial full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('. . .Alfred Bertrand.').toString(),
+        '\u2026Alfred Bertrand.'
+      )
+    }
+  )
+
+  await t.test(
+    'should replace three final full stops with spaces',
+    async function () {
+      assert.equal(
+        processor.processSync('Alfred Bertrand. . .').toString(),
+        'Alfred Bertrand\u2026'
+      )
+    }
+  )
+})