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

[Bug] Ascending/descending sorting is opposite to the one expected #669

Closed
1 task done
Currrupted opened this issue Nov 18, 2024 · 2 comments · Fixed by #672
Closed
1 task done

[Bug] Ascending/descending sorting is opposite to the one expected #669

Currrupted opened this issue Nov 18, 2024 · 2 comments · Fixed by #672
Labels
priority: high High priority type: bug Something isn't working
Milestone

Comments

@Currrupted
Copy link

Description

In the lists and browser of tracks, albums, artists etc., having both an ascending and a descending order results in a list having the order in reverse of what is to be expected by the respective icon (the arrow).

Showing all albums (in the respective tab), sorted by their track count in an ascending order (an arrow turned upwards), results in the albums with the highest number of tracks at the top. Respectively, sorting all tracks by year in an descending order (as indicated by an arrow turned upwards) results in a list going from the tracks with the lowest year/number to the highest.

This seems to be the case in all of the tabs that you can go to to: "Songs", "Albums", "Artists", "Playlists", "Album artists", "Genres".

Steps to Reproduce

  1. Select and go to any given tab noted above
  2. Select a field/attribute to sort by
  3. Select the arrow for an ascending order (turned upwards)
  4. Check the value for the attribute of the element at the top
  5. Check the value for the attribute of the element at the bottom
  6. Select the arrow for an descending order (turned downwards)

Expected Behavior

Sorting by an ascending order should show first element at the top/beginning. Sorting by an descending order should show the element, that is last in the chain, in the first spot. An arrow turned downwards should show an descending arrow, and vice versa.

Screenshots

No response

Additional Context

No response

Device

Sony Xperia XA2

OS

LineageOS 21 (Android 14)

Version

2024.11.114

Contribution Guidelines

@Currrupted Currrupted added the type: bug Something isn't working label Nov 18, 2024
@sevonj
Copy link
Contributor

sevonj commented Nov 18, 2024

This seems obvious now but I didn't even notice while using the app.

This also applies to alphabetical sorting.

@zyrouge
Copy link
Owner

zyrouge commented Nov 18, 2024

Yup, I did recently notice it too. Thanks for reminding it.

@zyrouge zyrouge added the priority: high High priority label Nov 18, 2024
@zyrouge zyrouge added this to the v2024.12.x milestone Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high High priority type: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants