Fix ordinal superscript for numbers ending in 0 (20th, 30th, etc.) (#3241)

- Changed falsy check to explicit !== null for numericValue
- Fixes issue where 0 was treated as falsy, preventing superscript
- Affects both separate segments (linked numbers) and combined (20th) formats
- Added unit tests for 20th, 30th, and 40th to prevent regression

Author: gabesl

packages/roosterjs-content-model-plugins/lib/autoFormat/numbers/transformOrdinals.ts

@@ -40,15 +40,15 @@ const ORDINAL_LENGTH = 2;
             if (
                 numberSegment &&
                 numberSegment.segmentType == 'Text' &&
-                (numericValue = getNumericValue(numberSegment.text, true /* checkFullText */)) &&
+                ((numericValue = getNumericValue(numberSegment.text, true /* checkFullText */)) !== null) &&
                 getOrdinal(numericValue) === value
             ) {
                 shouldAddSuperScript = true;
             }
         } else {
             const ordinal = value.substring(value.length - ORDINAL_LENGTH); // This value  is equal st, nd, rd, th
             const numericValue = getNumericValue(value); //This is the numeric part. Ex: 10th, numeric value =
-            if (numericValue && getOrdinal(numericValue) === ordinal) {
+            if (numericValue !== null && getOrdinal(numericValue) === ordinal) {
                 shouldAddSuperScript = true;
             }
         }

packages/roosterjs-content-model-plugins/test/autoFormat/numbers/transformOrdinalsTest.ts

@@ -142,6 +142,48 @@ describe('transformOrdinals', () => {
         runTest(segment, paragraph, { canUndoByBackspace: true } as any, true);
     });
 
+    it('with 20th', () => {
+        const segment: ContentModelText = {
+            segmentType: 'Text',
+            text: '20th',
+            format: {},
+        };
+        const paragraph: ContentModelParagraph = {
+            blockType: 'Paragraph',
+            segments: [segment],
+            format: {},
+        };
+        runTest(segment, paragraph, { canUndoByBackspace: true } as any, true);
+    });
+
+    it('with 30th', () => {
+        const segment: ContentModelText = {
+            segmentType: 'Text',
+            text: '30th',
+            format: {},
+        };
+        const paragraph: ContentModelParagraph = {
+            blockType: 'Paragraph',
+            segments: [segment],
+            format: {},
+        };
+        runTest(segment, paragraph, { canUndoByBackspace: true } as any, true);
+    });
+
+    it('with 40th', () => {
+        const segment: ContentModelText = {
+            segmentType: 'Text',
+            text: '40th',
+            format: {},
+        };
+        const paragraph: ContentModelParagraph = {
+            blockType: 'Paragraph',
+            segments: [segment],
+            format: {},
+        };
+        runTest(segment, paragraph, { canUndoByBackspace: true } as any, true);
+    });
+
     it('with 11th', () => {
         const segment: ContentModelText = {
             segmentType: 'Text',