Entertainment-web-app

Community feedback

• 

  Anton•310

  @ShevchukAPosted over 1 year ago

  Awsome work! I have complited the same challenge on this week and now I see how far is my variant from your solution :) Trending line scrolling with mouse drag is cool, also I really like dummy imagies while loading process is going. If you don't mind I would like to share my suggestions for improvements:

  • a little bit strange behaviour with bookmark button - I think it's better to UX to swich state immidiatelly and send request in a "background".
  • check the behaviour of trending line when resizing... there are some twitch.
  • improve responsive behaviour with grid smooth resizing.

  Good luck! Great job!

  Marked as helpful

  0

  ---

  

  Md Ashfaque•280

  @mdashfaque-mamdu-au17Posted over 1 year ago

  @ShevchukA Thanks for the feedback it means a lot to me. I will work on the things you mentioned.

  0

  

  Anton•310

  @ShevchukAPosted over 1 year ago

  @mdashfaque-mamdu-au17 you are welcome) For the hover state styling you can use importing svg as a react component. It allows you to get access to 'fill' attribute: import { ReactComponent as IconHome } from "../../assets/icon-nav-home.svg";

  svg:hover path {
  fill: red;
  }
  

  0

  

  Md Ashfaque•280

  @mdashfaque-mamdu-au17Posted over 1 year ago

  @ShevchukA Thanks I will try that

  0