Ashinch / ReadYou

An Android RSS reader presented in Material You style.
GNU General Public License v3.0
5.14k stars 203 forks source link

feat(greader): sort categories alphabetically during sync #700

Closed mbestavros closed 5 months ago

mbestavros commented 6 months ago

This has been a pet peeve of mine when using the Greader API: categories/folders aren't sorted alphabetically when in the Feeds view. The way the app works today, categories are arranged by whichever category has the feed with the lowest lexicographical order, which is very unintuitive.

As an example, let's say I have the following categories and feeds:

"Apples": {
    "feeds": ["Fruits of the Day"]
},
"Bananas": {
    "feeds": ["A banana a day keeps the doctor away"]
},

Under the current code, these categories get ordered as:

Bananas
Apples

because A banana a day keeps the doctor away is lexicographically before Fruits of the Day.

While it's true that we should probably implement some way to allow the user to manually reorder feeds, this is a quick fix that requires little effort. Plus, even if we did have manual category sorting, it's still nice to present a sensible lexicographic default.

Tagging @Ashinch and @JunkFood02 for review.

mbestavros commented 6 months ago

Related: #535