Ecommerce product page main

Community feedback

• 

  Azizbek Yunusaliev•400

  @azick99Posted over 1 year ago

  Well-done Hoda! I see you did a lot and I like your css code.

  However, I got some advice to improve your code and user-experience.

  1. Do not use width: 1160px; it can look like well in your PC but user will have different screens that developer must consider. Try to find better alternative like width: 90vw; . Bonus for user-experience make border of image avatar: instead of .user-profile{ height: 3.8rem; cursor: pointer; } .user-profile:hover{ border: 2px solid #f98001; border-radius: 50px; }make .user-profile{ height: 3.8rem; cursor: pointer; border: 2px solid #fff; border-radius: 50px; }then hover: {border-color:#f98001}.
  2. In page you got a bug in functionality, when you open dropdown and at the same time modal popup close disappears under dropdown, and you can close the modal. To solve it make z-index of popup higher and dropdown lower.
  3. Try to improve your JS code, there are too many repetitions, work with functions that handles most problems.

  I hope it would be useful. Keep coding!!!

  Marked as helpful

  0

  ---

  

  Hoda•140

  @Hoda96Posted about 1 year ago

  hello dear@azick99 thanks for your attention and advises. I`m new to js and really need these kind of tips. thanks again.

  0