Three Column Preview Component ( HtML and SCSS)

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <div class="container"> with the main tag to fix the accessibility issue. The body is missing a background-color that it must have.

  No need for the <br> increase the padding value for each box. Reduce the width of the buttons and increase the padding-top and bottom values.

  Ther is no need to give .container a max-width height value. change the max-width value to a fixed value max-width: 900px instead of a percentage value.

  In the media query, reduce the max-width of .container max-width: 400px and give it a flex-direction of column. The rest are not needed. There is no need to give each box a margin-top value.

  Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units watch this https://youtu.be/N5wpD9Ov_To

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  Marked as helpful

  0