Skip to content

fix(library): normalize artist name sorting across the API and UI#175

Open
cachamber wants to merge 2 commits intoChevron7Locked:mainfrom
cachamber:artist_sort
Open

fix(library): normalize artist name sorting across the API and UI#175
cachamber wants to merge 2 commits intoChevron7Locked:mainfrom
cachamber:artist_sort

Conversation

@cachamber
Copy link
Copy Markdown
Contributor

Description

This PR fixes artist sorting behavior in the library:

artist names now sort without case sensitivity
names beginning with “The ” sort by the next word
server-side ordering now matches UI behavior and is applied before pagination

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Enhancement (improvement to existing functionality)
  • Documentation update
  • Code cleanup / refactoring
  • Other (please describe):

Related Issues

Fixes #174

Changes Made

  • Backend: updated artist list sorting in [artists.ts]
  • Frontend: normalized artist sorting for library view in [page.tsx]

Testing Done

  • Tested locally with Docker
  • Tested specific functionality:
    • Lower case artist "alt-J" sorted into the As
    • "The Beatles" sorted into B

Screenshots (if applicable)

Before:
Screenshot 2026-04-03 200007
After:
Screenshot 2026-04-03 201321
Screenshot 2026-04-03 201339

Checklist

  • My code follows the project's code style
  • I have tested my changes locally
  • I have updated documentation if needed
  • My changes don't introduce new warnings
  • This PR targets the main branch

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.

[Bug]: Artist sorting should not be case sensitive - also ignore "The" in sorting

1 participant