-
Notifications
You must be signed in to change notification settings - Fork 68
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
Utilize favorites feature elsewhere in app #984
Utilize favorites feature elsewhere in app #984
Conversation
d6ee89e
to
5cac355
Compare
This is awesome, thanks for doing this! It'll really help bring some QoL improvements to the favourites action! I do have one minor discussion point around UI/UX. This is with regards to the favorites button being on the app bar for a community. I think Material design generally only recommends having at most 3 actions on the app bar. For more than 3 actions, we would use a dropdown menu to show the additional actions. This is the guidelines: https://m3.material.io/components/top-app-bar/guidelines#224e8ca0-4177-45f4-9f4f-eef1fbb0e4b2 Now, I don't know if we necessarily need to strictly follow their guidelines, which is why I want to bring up this discussion around this. If we do want to follow their guidelines, then we need to think about which actions should be placed in the dropdown menu! Otherwise, we could leave this for a future discussion (when we eventually have more actions that need to be placed at the top) |
That's a good point, especially since it would truncate the community name even further. What do you think about putting Favorite (since it's new) and Refresh (since it's available via FAB) in the overflow? |
Yeah, I think that sounds good! |
Great, should be all set now! On the main feed, refresh and sort are visible. In a community, subscribe and sort are visible, while refresh and favorite are in the overflow! qemu-system-x86_64_5u3ORzDFzR.mp4Note Review without whitespace now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Pull Request Description
This PR adds a few enhancements on top of the recent favorite communities feature.
feed.mp4
input.mp4
search.mp4
search_result.mp4
EDIT: I added another commit which prioritizes favorites in the list of communities shown in the input dialog before the user has typed anything.