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

Feed list not ordered by unread count anymore #697

Open
Thor77 opened this issue Jun 20, 2020 · 4 comments
Open

Feed list not ordered by unread count anymore #697

Thor77 opened this issue Jun 20, 2020 · 4 comments

Comments

@Thor77
Copy link

Thor77 commented Jun 20, 2020

It seems like the latest release, or rather this commit 4cdfd64, changed the ordering of the feed list.
The feeds are not ordered by unread count anymore but just by errors/title now.

Is there a reason for this change and are there any plans to revert it/make it configurable?

@fguillot
Copy link
Member

The goal of this commit is to render the feeds page faster. The subqueries to fetch counters is too slow. I haven't found a way to keep the unread count order. Feel free to propose a better solution.

@shizunge
Copy link
Contributor

@Thor77 Would you like to have look at #668
I am trying to add sorting to the feed list.

It is currently intentionally done on the client side, which also means if you always want to sort by unread, you need a few clicks every time visiting the feed page.

I can see what I can do, if you want to keep the sorting choice permanently.

@Thor77
Copy link
Author

Thor77 commented Jun 27, 2020

@shizunge yes, both of your PRs would cover my usecase perfectly, thanks!
It would be nice to have an option to set the default sort key in profile settings (or just remember the last selected one), but that's not strictly required imo.

@ldexterldesign
Copy link

ldexterldesign commented Jan 14, 2024

👋 #1434

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

4 participants