-
Notifications
You must be signed in to change notification settings - Fork 44
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
Video category page design #1133
Comments
@bedeho Here's the prototype to check out the new iteration for this page, and a video update to learn all about it.
Please, let me know if you have any questions, ideas, or other things you'd like to discuss. Once this is approved, I can proceed with RWD and preparations for the developer handoff. :) |
NoteI would actually have preferred to provide this feedback directly on the Figma designs somehow, because it makes it easier to describe what I am referring to. Please let me know if this still is possible. Feedback.This looks really promising and the video made it very to understand the full design. The subject of this feedback is only the view of each individual category Here are some observations.
|
@bedeho Thanks for the feedback! Here's a prototype with the new iteration, and a video update with my commentary.
This time, apart from your feedback around the iteration as a whole, I'll need your explicit decision on which direction you'd like to proceed with in regards to filters in the "All videos" section:
|
Amazing Option summary, video and work <3. I think Option A is best, but do we really need an |
@bedeho Thanks! I included the "Apply" button because we decided to keep filters separate boxes (popovers). As a consequence, the "Apply" button serves here as a closure: not only does it let you close the filters popover, but it also reassures you about your filtering actually being applied. I hope this makes sense.
And while I'm mentioning this throughout the documentation in files listed above, please keep in mind that implementing the new Video category page requires implementing new or adjusting existing components:
Ideally, we can sync together before you start working on this, just like we did with the Discover page. Let me know when you're available. |
The scope of this issue is to review, update and finalize the designs of the Discover page, formerly created by @KubaMikolajczyk, based on the feedback from Bedeho in #874 and the research done by @KubaMikolajczyk.
This includes:
The text was updated successfully, but these errors were encountered: