Mobile-first solution using Flexbox and CSS Grid

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 3 years ago

  Hi Liandro

  This looks really nice and the html is well structured.

  I recommend removing the picture elements from your html - they are only useful if you need to serve multiple image resolutions not a single tiny svg.

  Also, try to remove all the heavy nesting in your css. Keeping specificity as low as possible is best practice and and nesting element selectors like this main section picture img will cause real problems if you get into that habit now and come to bigger projects later.

  Hope these pointers help you. I look forward to seeing your next solution

  2

  ---

  

  Leandro•120

  @LeandroRicoPosted over 3 years ago

  Hi Grace!

  Wow, i didn't know that about the picture elements.

  And you are right about specificity, i totally forgot that, but i'll keep in mind for the next projects.

  Thanks for your feedback! 😊

  0
• 

  Aman Deep Bisht•200

  @aman152003Posted over 3 years ago

  Great Work, I also submitted the same solution but your looks better 😉

  1

  ---

  

  Leandro•120

  @LeandroRicoPosted over 3 years ago

  Hi aman! Thanks! I'm glad you like it 😊

  0