Multipage web app built with React and TailwindCSS

Community feedback

• 

  Francisldn•250

  @francisldnPosted over 2 years ago

  Hi @ericnbello great job on completing this challenge. I have just submitted my solution today for this challenge. I have a few comments on your solution:

  • I notice that you used React and TailwindCSS for your solution. Similarly I have use these technologies, I'm a big fan of both React and Tailwind.
  • On large screen size, the spaces on margin left and right seem quite large, I would try to adjust the spacing for the layout on large screen size.
  • For the "Trending" movies, I personally think that the scrollbar is not aethestically pleasing, so I chose to get rid of it using a package called "tailwind-scrollbar-hide", you can check it out.
  • The bookmark functionality works when user clicks on it but it doesn't add to the "Bookmark" page when I bookmark a movie/TV show.
  • On the "Bookmark" page, when I remove a bookmark, I would expect the movie/TV show to drop off from the list, but it remains on the list.
  • Good job on implementing the search functionality.

  Marked as helpful

  0

  ---

  

  Eric Bello•300

  @ericnbelloPosted over 2 years ago

  @francisldn Thanks for the detailed feedback! I'm going to revisit this project and implement each of these points you made because they are important from a user perspective but I overlooked them.

  0