search

axiel7 / AniHyou-android

Another unofficial Android AniList client
https://axiel7.github.io/anihyou/
GNU General Public License v3.0
324 stars 6 forks source link

Sorting by Title/Popularity #3

Closed Swadhin05 closed 4 months ago

Swadhin05 commented 1 year ago

Request (required) I personally prefer anime/manga list in alphabetical order. Pls add it

Examples

IMG_20230602_114554

axiel7 commented 1 year ago

There's currently a bug in the AniList API that returns wrong data when sorting by title. I plan to add this option but I'll need some time to think a workaround or wait until AniList fix it

texastoland commented 11 months ago

Still not working? Title is my preferred sort but Release Date would be nice too 🙏🏼

axiel7 commented 11 months ago

It seems AniList fixed it so I'll add it back

axiel7 commented 11 months ago

AniList haven’t completely fixed the sort bug, so sorting by title and popularity shows the Mean Score instead of your score and incorrect status.

I won’t remove title sorting again, let’s wait and hope that AniList fixes it someday

texastoland commented 11 months ago

I came to report the same thing. If it's a maximum number of pages of results (5 for example) could you implement sorting client side? https://github.com/axiel7/AniHyou-android/blob/afc0aa0c7cfd5eabb4e9ae2f3fd37af9f0be28b0/app/src/main/graphql/operations/medialist/UserMediaList.graphql#L1-L3

axiel7 commented 11 months ago

If it's a maximum number of pages of results (5 for example) could you implement sorting client side?

I could do it but as you said it would only work fine with small lists so there's no point.

I recommend to use only those sorting options for browsing your list. If you wanna make edits, change the sort or open the details screen

texastoland commented 11 months ago

I CCed the maintainer (I think) on Discord maybe they can help 🤞🏼

axiel7 commented 7 months ago

It seems AniList changed dev teams, let's hope they fix this API bug soon.

Screenshot 2024-04-02 at 13 08 06
akyraisconfused commented 5 months ago

sorting by title works fine on other clients so i was wondering if the api just dislikes your implementation or something (not that i understand this stuff)

axiel7 commented 5 months ago

sorting by title works fine on other clients

Probably they're doing some workaround. Could you give me an example?

akyraisconfused commented 5 months ago

im trying out a bunch of clients so of course! AL-chan, Aluminium, Anichive, AniLib, Kurumi and Otraku all have working alphabetical sorting.

texastoland commented 5 months ago

I thought it was a temporary problem but switching category (like planning vs watching) is currently broken (wrong grouping) too.

akyraisconfused commented 5 months ago

yeah, it sorts by title or popularity just fine but loads the wrong list

axiel7 commented 5 months ago

It seems most of those clients are sorting the list locally instead of sending the sort param to the AniList API.

Personally I prefer to send the sort param to the API because:

  1. It's simpler
  2. It leverages the processing work for sorting the list to the servers instead of your device
  3. The app doesn't need to load your entire list

For now I'll wait a bit more for AniList to fix this bug since they changed the dev team recently. But if they don't fix it in the next site update, I'll think about doing some workaround like that

akyraisconfused commented 5 months ago

yeah, i'd guessed it had something to do with client-side vs server-side. keep up the good work ^-^

axiel7 commented 4 months ago

Finally fixed this in 1.3.3