flexbox

Community feedback

• 

  romila•3,570

  @romila2003Posted over 2 years ago

  Hi Jorge,

  Congratulations 🎉 on completing this challenge, it was a great attempt.

  When looking at your design, I noticed the lack of responsiveness when shifting from 501px to around 907px. The reason for this is most likely because of your approach for your media queries. It is best practice to use the mobile-first approach since it would be easier to adjust the elements when increasing the screen size. So, instead of giving your body a property of media screen and (max-width: 500px), you should use the min-width property instead as it will be easier to control the container.

  Another minor issue is your background-color, I think you forgot to set it as the var(--Very-light-gray) style, for your body.

  Overall, good attempt and wish you all the best for your future projects 👍.

  Marked as helpful

  0