Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browse/Filesystem function sorts lowercase letters after all uppercase, not alphabetically #755

Closed
FrankJaco opened this issue Jul 28, 2022 · 1 comment · Fixed by music-assistant/server#451
Labels
enhancement Improvements of existing functionality

Comments

@FrankJaco
Copy link

What version of Music Assistant has the issue?

2022.7.2

The problem

A very minor issue, Marcel, but I wanted to report it.

When in Browse/Filesystem the folders are displayed in alphabetical order as it should, unless there is a mix of folders starting with uppercase letters and lowercase. MA will first display all the folders starting with "A" to "Z" then will display folders "a" - "z".

image

It is an issue that is easy to work around... Simply rename your lowercase folders with uppercase letters, and all is good.

How to reproduce

To reproduce change a few folder names in the filesystem to start with lowercase letters, then Browse/filesystem again.they will immediately show up after all of the uppercase entries.

Relevant log output

No response

Additional information

Low priority for sure.

What version of Home Assistant Core are your running

2022.7.6

What type of installation are you running?

Home Assistant OS

On what type of hardware are you running?

Generic x86-64 (e.g. Intel NUC)

@OzGav
Copy link
Contributor

OzGav commented Jul 30, 2022

Yes Marcel did some work previously with sorting and my instinct was that what you are observing is correct as in ASCII order that is what you would see. However, the yardstick we used at the time of the previous fix is what would Excel do. I just tried it and indeed Excel sorts as you are suggesting so I will mark this as an enancement and see what Marcel thinks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants