Mobile first and Flexbox

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Nice work, Herman! Your solution looks really good and it's great to see you using Flexbox. Here are a couple of pointers after taking a look at your code:

  • You've got width: 750px; on the .container class. This is fine, but it means that you get a horizontal scrollbar on screen sizes smaller than 750px. You could add a max-width: 95%; to that class to get it to scale down when the screen size gets too small. You'd also then need to add margin: 0 auto to center it.
  • You've also got width: 375px; on the inner elements. Instead of using pixels for widths like this, I'd recommend using percentages wherever possible. This will help with building responsive websites as the elements will scale up and down with the screen.

  I hope that helps. Keep up the great work!

  0