Responsive layout using flex and media queries

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 5 months ago

  Hello there!

  Congrats on completing the challenge! ✅

  Your solution looks great!

  I have a suggestion for improvement:

  📌 Think about using <main> to wrap your main content instead of <div>.

  Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

  This change might not have impact on how your page looks, but it'll make your HTML code clearer and help with SEO and accessibility.

  Hope that's helpful!

  Keep up the great work!

  Marked as helpful

  1

  ---

  

  Attoungbre Ange François•160

  @AngeATTPosted 5 months ago

  Thanks u for your good advices as always ! I did change the tag and used main for SEO purposes@danielmrz-dev

  0
• 

  P

  Tuna Erten•360

  @tunaertenPosted 5 months ago

  In terms of design, it is quite close to the original, but the fonts and colors are different from those specified. I don't think adding padding to the body in the CSS file is a good option; instead, I would add it to the necessary place (e.g., container or another element). I think I would use classes instead of IDs for larger projects because this way, I could use the same elements in multiple different places

  Marked as helpful

  1

  ---

  

  Attoungbre Ange François•160

  @AngeATTPosted 5 months ago

  Thanks for your reply, you are right i didnt use the right font color for the price i changed that@tunaerten

  0