Skip to content

Sort statistics by completion date - #19

Open
reilly1993 wants to merge 1 commit into
JenspederM:mainfrom
reilly1993:fix/statistics-date-order
Open

reilly1993 wants to merge 1 commit into
JenspederM:mainfrom
reilly1993:fix/statistics-date-order

Conversation

@reilly1993

@reilly1993 reilly1993 commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Completed games on the Statistics page appeared in an unintuitive order because they were sorted by solve time rather than completion date.

Solution

  • Sort each difficulty history from newest completion to oldest.
  • Keep the score array from context immutable.
  • Replace misleading podium/rank markers with a neutral calendar icon because the list is now chronological.
  • Add coverage for mixed dates, same-day times and source-array immutability.

Verification

  • Rebased onto the latest main.
  • 187 Bun tests pass.
  • TypeScript typecheck passes.
  • Biome passes across src.
  • Production Vite/PWA build and git diff --check pass.
  • Manually verified in the local Statistics UI.

@JenspederM

Copy link
Copy Markdown
Owner

Should be a selector. I go for time when looking at stats ;)

Signed-off-by: Reilly <christian@weld.app>
@reilly1993
reilly1993 force-pushed the fix/statistics-date-order branch from 41759ef to 1082215 Compare September 1, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants