Fix capitalization of acronym in sentence case

blakeembrey · Jan 8, 2024 · 59d762f · 59d762f
1 parent f7ece2c
commit 59d762f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/packages/title-case/src/index.spec.ts b/packages/title-case/src/index.spec.ts
@@ -135,7 +135,15 @@ const TEST_CASES: [string, string, Options?][] = [
     "Something About _Me-Too_? And You Too.",
   ],
   ["an example. i.e. test.", "An Example. I.e. Test."],
+  ["an example, i.e. test.", "An Example, I.e. Test."],
   ['an example. "i.e. test."', 'An Example. "I.e. Test."'],
+  ["an example. i.e. test.", "An example. I.e. test.", { sentenceCase: true }],
+  ["an example, i.e. test.", "An example, i.e. test.", { sentenceCase: true }],
+  [
+    'an example. "i.e. test."',
+    'An example. "I.e. test."',
+    { sentenceCase: true },
+  ],
 ];
 
 describe("swap case", () => {

diff --git a/packages/title-case/src/index.ts b/packages/title-case/src/index.ts
@@ -101,7 +101,10 @@ export function titleCase(
       // but we should uppercase first for i.e., e.g., etc.
       if (acronym) {
         const [_, prefix = "", suffix = ""] = acronym;
-        result += upperAt(token, prefix.length, locale);
+        result +=
+          sentenceCase && !isNewSentence
+            ? token
+            : upperAt(token, prefix.length, locale);
         isNewSentence = terminators.has(suffix.charAt(0));
         continue;
       }