Responsive page using Sass and JS

Community feedback

• 

  bimalmagar•380

  @bimalmagar10Posted over 3 years ago

  You really did great job building this site.Most of the part is all fine but what i found a bit unusual is:

  • When the window resizes to about 328px then the bookmark button overlaps the back this project button.You can fix that by using flex-direction:column.
  • Also, the out of stock button is clickable which seems totally unusal.You can fix it by setting pointer-events:none as well as cursor:not-allowed.

  0
• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Great job you got there. The layout differs in terms of sizing but it is fine for now.

  Layout also resizes good when changing to mobile state view. However, one issue that I encountered is that, I am able to click the last options which should not be interactive right. Since it is "out of stock" user should not be able to click it.

  You could fix this by adding pointer-events: none to the selector of that button. That way, all the event will not be fired when a user clicks on it.

  Overall, your work is fine, good job^

  0