Responsive product preview card component

Community feedback

• 

  Precious Nkwor•440

  @Jeen-PreshPosted almost 2 years ago

  Hello Ryan, Congratulations on completing the fem challenge

  Some suggestions to help improve your code

  • It's best practice to wrap your main content in a main element

  more info MDN main element

  • Best use the picture element to display different images for different display screens.

  more info MDN Picture elment

  • Your solution isn't responsive.

  more info MDN Resposniveness

  • Your desktop design is meant to have a two-column layout. Create a container and set the display property to the grid. and give a two-column layout.

  more info MDN Grid

  Happy coding. if you found this helpful, make sure to mark as helpful thanks.

  Marked as helpful

  0