-
Notifications
You must be signed in to change notification settings - Fork 739
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
Converge on Explore rooms view for Spaces #5658
Comments
After discussion with @niquewoodhouse and @kittykat it was decided to keep list item (room) design unchanged, since same layout is used for suggested room item in room list. Also search probably should be a part of toolbar to align with other screens with rooms filtering. New designs needed so reassign to @niquewoodhouse |
Thanks @fedrunov for notes above. Here's some designs with toolbar aligned with other screens, for more consistent experiences.
This would be consistent with the more prominent lists of rooms in the app, so people should know how to use it but it would mean finding the search is slightly harder. ele-android-explore-space-01.mov
This would be my preference but I don't feel super strongly, @fedrunov could you comment on which is easier to do/more feasible please? Thanks |
1st shouldn't be much harder to implement, and I also like it more than 2nd, so I vote for option 1 |
I don't feel super strongly either way - happy to go with 1 |
I took a look at the original designs and couldn't see any of the Android content at the top of explore, so in Figma they were always more similar to start with.
Small notes:
-
not be visible (see below)? It should also be a middle dot, not a dash.Figma Android
The text was updated successfully, but these errors were encountered: