Sunnyside landing page with HTML5, CSS3 Grid & Flexbox, Vanilla JS

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffePosted almost 4 years ago

  Hi, JuanCFS! 👋

  Good job on this challenge! 👏 Your solution looks nice and is responsive! 👍

  One thing I'd like to suggest is to use a single <h1> tag per page (since that’s commonly considered a good practice, now). In this case, the main heading ("We Are Creatives") could be the <h1> and some less important heading tags (such as <h2> or <h3>) should work well for the rest of the headings inside the other sections of the page. Just a tip! 😉

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  0

  ---

  

  JuanCFS•160

  @juancfsPosted almost 4 years ago

  Hi @ApplePieGiraffe!

  Thanks for your feedback. You are right, I think I should avoid using more than <h1>. I'll apply your recommendation. Thank you and happy coding, too!

  1
• 

  P

  Patrick•14,265

  @palgrammingPosted almost 4 years ago

  Nice Job ⭐⭐⭐⭐⭐

  Marked as helpful

  0