Skip to content

feat: randomize plot order on every page reload#3816

Merged
MarkusNeusinger merged 2 commits intomainfrom
claude/randomize-plot-order-y1zPl
Jan 12, 2026
Merged

feat: randomize plot order on every page reload#3816
MarkusNeusinger merged 2 commits intomainfrom
claude/randomize-plot-order-y1zPl

Conversation

@MarkusNeusinger
Copy link
Copy Markdown
Owner

Remove deterministic shuffle seed to ensure plots appear in a new
random order on each F5/reload, even when filters remain unchanged.

Changes:

  • Remove hashFilters() function (no longer needed)
  • Use Math.random() instead of seeded shuffle
  • Applies to both filtered and unfiltered views

Fixes random order behavior on homepage

Remove deterministic shuffle seed to ensure plots appear in a new
random order on each F5/reload, even when filters remain unchanged.

Changes:
- Remove hashFilters() function (no longer needed)
- Use Math.random() instead of seeded shuffle
- Applies to both filtered and unfiltered views

Fixes random order behavior on homepage
Clean up after removing deterministic shuffle:
- Remove seededRandom() function (no longer needed)
- Remove optional seed parameter from shuffleArray()
- Simplify to use Math.random() directly
- Update comment for clarity

No functional changes - just code cleanup.
Copilot AI review requested due to automatic review settings January 12, 2026 19:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes deterministic shuffling behavior to ensure plots appear in a new random order on every page reload, regardless of whether filters change.

Changes:

  • Removed seededRandom() function (mulberry32 PRNG implementation)
  • Removed hashFilters() function (used to generate deterministic seed from filter state)
  • Simplified shuffleArray() to always use Math.random() instead of optionally using a seeded RNG

@MarkusNeusinger MarkusNeusinger merged commit 6844f02 into main Jan 12, 2026
12 checks passed
@MarkusNeusinger MarkusNeusinger deleted the claude/randomize-plot-order-y1zPl branch January 12, 2026 19:56
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.

3 participants