Responsive site achieved using Flex box layouts & MediaQueries

Community feedback

• 

  Gracia Naimora Samosir•180

  @cia2003

  Posted 3 months ago

  Hello!

  I see that you want to put a creativity in your project, like using different color for the background. It is great!

  I suggest you read about BEM methodology to improve your code's clarity. It will be helpful for other developers to read your code. For example, I see in your html's code like this:

  <div class="price">
    <span class="pic1"> $149.99 </span>
    <span class="pic2"> $169.99 </span>
  </div>
  

  As you can see in the div.price there are two spans with class pic1 and pic2. It is unclear if you said it as a price or picture.

  For the product category (div.DIV1), you can use properties like text-transform' to uppercase the text or letter-spacing` to give space between the letters in CSS.

  Hope that helps!

  00