From 39d071aec67fd6c94f8a80eeaf7a3ff0b2e63c64 Mon Sep 17 00:00:00 2001 From: Scrum4Me Agent <30029041+madhura68@users.noreply.github.com> Date: Wed, 6 May 2026 07:17:09 +0200 Subject: [PATCH] feat(ideas): sorted useMemo + tabel-render koppelen (ST-cmotjj9uf000104l5i70so19b) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Voeg sorted useMemo toe na filtered: locale-aware sort met STATUS_SORT_ORDER - Ideeën zonder product sorteren achteraan bij oplopende productsortering - Vervang filtered.length/filtered.map door sorted in tabel-render Co-Authored-By: Claude Sonnet 4.6 --- components/ideas/idea-list.tsx | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/components/ideas/idea-list.tsx b/components/ideas/idea-list.tsx index e4a8dfe..d052fd1 100644 --- a/components/ideas/idea-list.tsx +++ b/components/ideas/idea-list.tsx @@ -154,6 +154,26 @@ export function IdeaList({ ideas, products, isDemo }: IdeaListProps) { }) }, [ideas, search, productFilter, statusFilter, sortKey, sortDir]) + const sorted = useMemo(() => { + return [...filtered].sort((a, b) => { + let cmp = 0 + if (sortKey === 'code') { + cmp = (a.code ?? '').localeCompare(b.code ?? '', 'nl', { numeric: true }) + } else if (sortKey === 'title') { + cmp = a.title.localeCompare(b.title, 'nl') + } else if (sortKey === 'product') { + const aN = a.product?.name ?? '' + const bN = b.product?.name ?? '' + if (!aN && bN) return sortDir === 'asc' ? 1 : -1 + if (aN && !bN) return sortDir === 'asc' ? -1 : 1 + cmp = aN.localeCompare(bN, 'nl') + } else { + cmp = (STATUS_SORT_ORDER[a.status] ?? 99) - (STATUS_SORT_ORDER[b.status] ?? 99) + } + return sortDir === 'asc' ? cmp : -cmp + }) + }, [filtered, sortKey, sortDir]) + function handleSort(col: SortKey) { if (sortKey === col) { setSortDir((d) => (d === 'asc' ? 'desc' : 'asc')) @@ -316,7 +336,7 @@ export function IdeaList({ ideas, products, isDemo }: IdeaListProps) { )} {/* Tabel */} - {filtered.length === 0 ? ( + {sorted.length === 0 ? (

{ideas.length === 0 ? 'Nog geen ideeën — start hierboven met "Nieuw idee".' @@ -334,7 +354,7 @@ export function IdeaList({ ideas, products, isDemo }: IdeaListProps) { - {filtered.map((idea) => { + {sorted.map((idea) => { const badge = getIdeaStatusBadge(API_TO_DB[idea.status]) return (