Skip to content

Commit 72c0606

Browse files
sdspiegclaude
andcommitted
Fix inverted date sort
byDate(dir)'s internal logic made dir=1 ascending (older→newer), not descending as the case statement assumed. Flipped the two case values so 'Date (newest)' actually shows newest first and 'Date (oldest)' shows oldest first. Confidence/speaker/target sorts were correct. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 41d8e34 commit 72c0606

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

src/components/Statements.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -160,10 +160,11 @@ export default function Statements() {
160160
};
161161
void at;
162162
switch (sortBy) {
163-
case 'date-desc': sorted.sort(byDate(1)); break;
164-
case 'date-asc': sorted.sort(byDate(-1)); break;
165-
case 'conf-desc': sorted.sort(byConf(-1)); break;
166-
case 'conf-asc': sorted.sort(byConf(1)); break;
163+
// byDate(dir): dir=1 means ascending (older→newer); dir=-1 means descending (newer→older)
164+
case 'date-desc': sorted.sort(byDate(-1)); break; // newest first
165+
case 'date-asc': sorted.sort(byDate(1)); break; // oldest first
166+
case 'conf-desc': sorted.sort(byConf(-1)); break; // high → low
167+
case 'conf-asc': sorted.sort(byConf(1)); break; // low → high
167168
case 'speaker': sorted.sort(byStr('speaker')); break;
168169
case 'target': sorted.sort(byStr('target')); break;
169170
}

0 commit comments

Comments
 (0)