CSS GRID, Flexbox, Media Queries

Community feedback

• 

  Kimo Spark•2,190

  @kimodev1990Posted 11 months ago

  Great work in completing the challenge, Just a few feedback :

  • To center your content in the middle of your website, you could add in body :

  display: flex ;
  justify-content: center ;
  align-items: center ;
  min-height: 100vh ;
  

  then the content will be centered, no need to use margin: 0 auto ;

  • Wrap your design in main tag.
  • You could use clamp ( ) method in your coding for font-size, width, margins, padding, etc., So the designed sizes will change according to the viewport dimensions having a responsive design and will be suitable for any device layout.

  Hope you find this Helpful.

  Other than that, Really Nice work & keep Going on

  Marked as helpful

  0

  ---

  

  Saúl Ventura•60

  @saulventuragalvezPosted 11 months ago

  @kimodev1990 Thank you for your feedback! I really appreciate the suggestion to center the content using flexbox and adjust the size with the clamp() method. I'll implement that right away.

  Thanks again for your help, and I appreciate your encouraging words! I'll keep working on improvements!

  0

  

  Kimo Spark•2,190

  @kimodev1990Posted 11 months ago

  @saulventuragalvez You're always welcome, Anytime....

  0