huddle landing page

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Congrats on completing the challenge successfully! Everything looks great.

  It looks like your background is repeating, causing unwanted content at the bottom, you could avoid that by using adding: background-repeat: no-repeat; in container.

  Also the footer icons look a bit far away from the main content, so you might consider having them a bit closer to the content.

  Hope you find this feedback helpful!

  Marked as helpful

  0
• 

  C H A R A N 🎯•2,670

  @0xabdulPosted over 1 year ago

  Hello there ! 👋 Congratulations you finished the Huddle landing page ...🎉

  • Some suggestions for you improve your code🤔

  Html 🏷️:

  • for Accessibility reports using the non-semantic elements (or) semantic elements..
  • semantic elements : <aside> , <artical> , <main>, <header> ,<section><footer>, <form> ect..
  • non- semantic elements : <div> , <span> ect ...
  • for easy way to clear the Accessibility reports using non semantic elements Ex :

  <div class="container" role="main">
  //Whole html code wraping the div tag named "container"..📍
  </div>
  

  • the html code should be contained one main landmark..❗
  • using the <h1> <h2> <h3> tags in order way , for Accessibility reports
  • I Hope it's useful for you and wating for your next project ❤️
  • Happy Coding 😃

  Marked as helpful

  0