fix: lowercase attendees before comparing so joined string matches pill order

Author: TaduJR

src/libs/TransactionUtils/index.ts

@@ -1220,8 +1220,9 @@ function getAttendeesDisplayCollator(): Intl.Collator {
  */
 function getAttendeesListDisplayString(attendees: Attendee[]): string {
     const collator = getAttendeesDisplayCollator();
+    // Lowercase to match sortAlphabetically (the pill sort) so joined string and pill order never disagree on case.
     return [...attendees]
-        .sort((a, b) => collator.compare(a.displayName ?? a.login ?? '', b.displayName ?? b.login ?? ''))
+        .sort((a, b) => collator.compare((a.displayName ?? a.login ?? '').toLowerCase(), (b.displayName ?? b.login ?? '').toLowerCase()))
         .map((item) => item.displayName ?? item.login)
         .join(', ');
 }

tests/unit/TransactionUtilsTest.ts

@@ -1408,6 +1408,14 @@ describe('TransactionUtils', () => {
             expect(TransactionUtils.getAttendeesListDisplayString(attendees)).toBe('User 9, User 10');
         });
 
+        it('compares case-insensitively so the joined string matches pill order', () => {
+            const attendees: Attendee[] = [
+                {email: 'b@x.com', displayName: 'Bob', avatarUrl: '', login: 'b@x.com'},
+                {email: 'a@x.com', displayName: 'alice', avatarUrl: '', login: 'a@x.com'},
+            ];
+            expect(TransactionUtils.getAttendeesListDisplayString(attendees)).toBe('alice, Bob');
+        });
+
         it('returns empty string for empty array', () => {
             expect(TransactionUtils.getAttendeesListDisplayString([])).toBe('');
         });