Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

support case-insensitive sorting #148

Open
teazdale opened this issue Apr 25, 2017 · 2 comments
Open

support case-insensitive sorting #148

teazdale opened this issue Apr 25, 2017 · 2 comments

Comments

@teazdale
Copy link

Description

Have a playlist where I am sorting by Album Artist (among others) and it appears where I have Album Artists with the first letter lowercase, will appear at the bottom of the playlist (see screenshot). I have other songs that are by the same artist(s) being sorted to the bottom, where the artist's name is Title-cased and it appears where I expected it to (in the alphabetical order). So it appears to be that the sorting algorithm being used is sensitive to case. Hope this wasn't too confusing. If you have any questions for me let me know. Hope the questions help.

Sorting Options selected

image

Album Artist in Question

untitled

Same Artist, But Sorted Differently

untitled2

P.S. - Dont judge my music taste and apologies if this is a non-issue.

@teazdale teazdale changed the title Sorting of Playlists seems to be case sensitive Sorting In Playlists Seems To Be Case Sensitive Apr 25, 2017
@simon-weber
Copy link
Owner

Ah, yes, it's case-sensitive. Behind the scenes this is done by https://github.com/google/lovefield, and it doesn't look like it supports case-insensitive sorts out of the box.

The simplest way to support this would probably be to just add extra columns with the data in lowercase. Or, if the column names are too tightly coupled to the output right now, I could always sort directly in javascript. Either way, though, I don't think I'll get to it anytime soon; sorry.

P.S. - Dont judge my music taste

Haha, no judgement here.

@simon-weber simon-weber changed the title Sorting In Playlists Seems To Be Case Sensitive support case-insensitive sorting Apr 25, 2017
@teazdale
Copy link
Author

@simon-weber

Thanks for the quick response! No problem on the delay, just noticed it and wanted to help out in any way I can. It is definitely not a deal breaker for me.

Good luck and keep up the good work!

@teazdale teazdale reopened this Apr 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants