All comments

• 

  Fathima Nuha•30

  @Fathima-NuhaSubmitted almost 2 years ago

  I've used the most updated knowledge I have learned so far! More to go! Have used flex, Do update me about places I could have used grid.

  

  Desktop solution using CSS Flexbox - Product preview card component

  4

  40
  

  AbdelRahman Elrefai•190

  @ar-refaiPosted almost 2 years ago

  Hello There Nice Work Fathima

  • I recommend that you consider adding <!DOCTYPE> declaration at the start of your HTML Document read about it from Click Here

  • also you should use css to change background color instead of using bgcolor attribute ,read HTML validation above.

  • I noticed that you use only <div> element in your HTML page , I Strongly recommend to wrap all the main content in a <main> element. Read More About Semantic Elements From Click Here

  • this is my solution for this challenge feel free to look at it and other solutions as well : Solution Link

  Keep Up The Good Work. I hope the comment was helpful

  Marked as helpful

  0

  This comment has received 2 repliesShow