From 45d7cd940dc80e6cdb6b8750dd7ab4f6087603ed Mon Sep 17 00:00:00 2001 From: Victor Giers Date: Sun, 1 Feb 2026 01:50:28 +0100 Subject: [PATCH] Simplify sorting logic in App.tsx --- src/App.tsx | 22 ++++++---------------- 1 file changed, 6 insertions(+), 16 deletions(-) diff --git a/src/App.tsx b/src/App.tsx index 430f8f3..0b1d3a3 100644 --- a/src/App.tsx +++ b/src/App.tsx @@ -182,18 +182,8 @@ export default function App() { const showLineNumbersActive = showLineNumbers && !markdownPreview; const hasSearch = search.trim().length > 0; - const sortByOrderThenUpdated = useCallback( - (a: T, b: T) => { - const aHasOrder = a.sort_order !== null && a.sort_order !== undefined; - const bHasOrder = b.sort_order !== null && b.sort_order !== undefined; - if (aHasOrder && bHasOrder) { - return (a.sort_order ?? 0) - (b.sort_order ?? 0); - } - if (aHasOrder !== bHasOrder) { - return aHasOrder ? -1 : 1; - } - return b.updated_at - a.updated_at; - }, + const sortByUpdated = useCallback( + (a: T, b: T) => b.updated_at - a.updated_at, [] ); @@ -217,10 +207,10 @@ export default function App() { } } for (const [key, list] of map.entries()) { - map.set(key, [...list].sort(sortByOrderThenUpdated)); + map.set(key, [...list].sort(sortByUpdated)); } return map; - }, [folders, sortByOrderThenUpdated]); + }, [folders, sortByUpdated]); const textsByFolder = useMemo(() => { const map = new Map(); @@ -234,10 +224,10 @@ export default function App() { } } for (const [key, list] of map.entries()) { - map.set(key, [...list].sort(sortByOrderThenUpdated)); + map.set(key, [...list].sort(sortByUpdated)); } return map; - }, [texts, sortByOrderThenUpdated]); + }, [texts, sortByUpdated]); const visibleFolderIds = useMemo(() => { if (!hasSearch) return null;