fix: Normalize literal before dayPeriod

Author: dklymenk

packages/@internationalized/date/src/DateFormatter.ts

@@ -34,7 +34,18 @@ export class DateFormatter implements Intl.DateTimeFormat {
 
   /** Formats a date to an array of parts such as separators, numbers, punctuation, and more. */
   formatToParts(value: Date): Intl.DateTimeFormatPart[] {
-    return this.formatter.formatToParts(value);
+    const parts = this.formatter.formatToParts(value);
+
+    const literalBeforeDayPeriodIndex =
+      parts.findIndex((p) => p.type === "dayPeriod") - 1;
+    if (literalBeforeDayPeriodIndex >= 0) {
+      const normalizedParts = parts.map((p, i) =>
+        i === literalBeforeDayPeriodIndex ? { ...p, value: " " } : p,
+      );
+      return normalizedParts;
+    }
+
+    return parts;
   }
 
   /** Formats a date range as a string. */

packages/@internationalized/date/tests/DateFormatter.test.js

@@ -42,6 +42,17 @@ describe('DateFormatter', function () {
     ]);
   });
 
+  it('should format to parts with space as literal before am/pm', function () {
+    let formatter = new DateFormatter('en-US', {timeStyle: 'short'});
+    expect(formatter.formatToParts(new Date(2020, 1, 3, 13, 0))).toEqual([
+      {type: 'hour', value: '1'},
+      {type: 'literal', value: ':'},
+      {type: 'minute', value: '00'},
+      {type: 'literal', value: ' '},
+      {type: 'dayPeriod', value: 'PM'}
+    ]);
+  });
+
   it('should format a range', function () {
     let formatter = new DateFormatter('en-US');
     // Test fallback