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

Default Sort setting doesn't include "Scaled" #1456

Closed
zevdg opened this issue May 3, 2024 · 3 comments · Fixed by #1471
Closed

Default Sort setting doesn't include "Scaled" #1456

zevdg opened this issue May 3, 2024 · 3 comments · Fixed by #1471
Labels
bug Something isn't working

Comments

@zevdg
Copy link

zevdg commented May 3, 2024

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

  1. Go to Settings
  2. Click on "Default Sort"
  3. See that "Scaled" is not listed as an option.

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
image

tested on both desktop and mobile

Desktop:

  • OS: ChromeOS
  • Browser: Chrome
  • Version: 2.7.1

Smartphone:

  • Device: Pixel 7 Pro
  • OS: Android
  • Browser: App
  • Version: 2.7.1
@zevdg zevdg added the bug Something isn't working label May 3, 2024
@aeharding
Copy link
Owner

Hi! I think you are viewing the default comment sort. Double check and let me know!

@zevdg
Copy link
Author

zevdg commented May 6, 2024

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)

  1. Change the label from "Default Sort" in both sections to "Default Post Sort" and "Default Comment Sort".
    Add a test ensuring that no 2 settings on the same page have exactly the same label text. "Default Sort" seems to be the only setting that appears twice in different sub-sections, but I would expect any 2 settings with the same exact name to cause confusion.

  2. Put each setting subsection into an "accordion" that are all collapsed by default. Having the user actively tap into the "Posts" or "Comments" subsections would anchor the context in their user journey.
    Optionally, one could also configure the accordions so that only 1 would be expanded at any time, which would be another way of ensuring that 2 settings with the same name are not ever displayed on the screen simultaneously.

@aeharding
Copy link
Owner

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants