Browse videos app - React, SCSS

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 2 years ago

  Excellent work on this challenge! You've hit the nail on the head with the design comparison. I like the subtle animations you've added as well. Here are a couple of small pieces of feedback:

  • If you scroll down on a page and then navigate to a new one, the screen remains halfway down the page. I'd expect it to pop up to the top when navigating to a new page.
  • The items are duplicated between the Movies and TV Series sections on the Bookmarks page. So there are movies showing in the TV series section and vice versa. You could use .filter() instead of map to return only the items of a given category.

  Amazing work overall. Keep it up! 👍

  Marked as helpful

  2

  ---

  

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 2 years ago

  Oh, and I'd recommend trying to resolve the accessibility issues in the report as well 🙂

  0

  

  Benja.min•740

  @BenjaDotMinPosted over 2 years ago

  @mattstuddert Appreciate the feedback! I will surely fix the scroll issue as that is easy. But I will dive into the others when I find some free time!

  Thank you again for your time.

  2