Skip to content

fix(web): drop count sort order from grouped result links - #92

Merged
andykais merged 1 commit into
mainfrom
cursor/group-count-sort-links-1356
Oct 13, 2026
Merged

andykais merged 1 commit into
mainfrom
cursor/group-count-sort-links-1356

Conversation

@andykais-ai

Copy link
Copy Markdown
Collaborator

Summary

  • improves grouped result links on /browse: when grouping is sorted by count, the link to the plain media search now drops order along with sort, so it opens with no explicit sort (e.g. ?tags=artist:bob instead of ?tags=artist:bob&order=asc)
  • improves group preview thumbnails: when sorted by count, the preview's media search no longer hardcodes created_at and uses core's default sort, matching the page the link opens
  • non-count sorts are unchanged (e.g. sort=view_count&order=asc still carries through to the link)

Vision Testing

  • seeded a local db with artist:alice (3), artist:bob (2) and artist:carol (1), then ran forager gui
  • /browse?group_by=artist&sort=count&order=asc: the bob link is ?tags=artist:bob and opens 2 results sorted by Created At (desc)
  • /browse?group_by=artist&sort=count: the alice link is ?tags=artist:alice and opens 3 results
  • /browse?group_by=artist&sort=view_count&order=asc: the alice link keeps sort=view_count&order=asc
  • deno task --cwd packages/web build:local succeeds

group-count-sort-links.mp4

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Co-authored-by: kaisea.rpi <kaisea.rpi@gmail.com>
@andykais
andykais marked this pull request as ready for review October 13, 2026 21:34
@andykais

Copy link
Copy Markdown
Owner

This may be a duplicate of 2ad88f1, and I was testing on an older version, but its still a reasonable change. Merging

@andykais
andykais merged commit 7c21fc9 into main Oct 13, 2026
8 checks passed
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