-
-
Notifications
You must be signed in to change notification settings - Fork 168
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
Default Sort setting doesn't include "Scaled" #1456
Comments
Hi! I think you are viewing the default comment sort. Double check and let me know! |
Doh, you're totally correct. Feel free to close this issue, and sorry for the noise! That said, consider repurposing this issue as a feature request (not a bug) to make the UX clearer, as other people may be making the same mistake. Here's 2 UX suggestions that would help (either one alone would resolve this issue IMO)
|
Good to hear! And thanks for the potential ways to make it clearer. I'll add "post" and "comment" to title when action sheet is open to select the sort. However I want to avoid adding it to the settings list for now (it will make things more wordy/redundant, and while it does fit, it probably won't for some other labels). However if others are also confused I'll try to further improve this. :) Thanks! |
Describe the bug
Several new sorting methods were added in lemmy v0.19
PR #838 added them to voyager, but it only added one of them (controversial) to the default sort setting for some reason.
To Reproduce
Expected behavior
Scaled should be a default sort option.
Time-based Top sorts should probably be supported as well, but I care less about that.
Screenshots
tested on both desktop and mobile
Desktop:
Smartphone:
The text was updated successfully, but these errors were encountered: