From dd0de3a50cee94dfb04424bf9219fae759fcdff2 Mon Sep 17 00:00:00 2001 From: DIYgod Date: Mon, 12 Oct 2026 21:59:56 +0800 Subject: [PATCH] fix(desktop): sort subscriptions A to Z for alphabetical ascending "Alphabetical, ascending" listed folders and feeds from Z to A and "descending" from A to Z, because the A-to-Z comparison was reversed for the ascending order. The unread count orders already matched their labels. Co-Authored-By: Claude Opus 5.5 --- .../src/modules/subscription-column/SortedFeedItems.tsx | 3 ++- .../subscription-column/sort-by/SortByAlphabeticalList.tsx | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/apps/desktop/layer/renderer/src/modules/subscription-column/SortedFeedItems.tsx b/apps/desktop/layer/renderer/src/modules/subscription-column/SortedFeedItems.tsx index 4923ead9d..76e543bab 100644 --- a/apps/desktop/layer/renderer/src/modules/subscription-column/SortedFeedItems.tsx +++ b/apps/desktop/layer/renderer/src/modules/subscription-column/SortedFeedItems.tsx @@ -58,7 +58,8 @@ const SortByAlphabeticalList = (props: SortListProps) => { const feedTitleB = getPreferredTitle(state.feeds[b]) || "" return sortByAlphabet(feedTitleA, feedTitleB) }) - return isDesc ? sorted : sorted.reverse() + // sortByAlphabet orders A to Z, which is ascending + return isDesc ? sorted.reverse() : sorted } // Return onboarding feeds first, then regular feeds diff --git a/apps/desktop/layer/renderer/src/modules/subscription-column/sort-by/SortByAlphabeticalList.tsx b/apps/desktop/layer/renderer/src/modules/subscription-column/sort-by/SortByAlphabeticalList.tsx index 7b519fc58..ed8cc6520 100644 --- a/apps/desktop/layer/renderer/src/modules/subscription-column/sort-by/SortByAlphabeticalList.tsx +++ b/apps/desktop/layer/renderer/src/modules/subscription-column/sort-by/SortByAlphabeticalList.tsx @@ -101,7 +101,8 @@ export const SortByAlphabeticalFeedList = ({ } return sortByAlphabet(nameA, nameB) }) - return isDesc ? sorted : sorted.reverse() + // sortByAlphabet orders A to Z, which is ascending + return isDesc ? sorted.reverse() : sorted } // Return onboarding categories first, then regular categories