sass htlm5 css3 grid flexbox res

Community feedback

• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted over 2 years ago

  If you want to reduce your accessibility, Change this code. 😊

  <div class="container"> to <main> ( Used main tag for main card design ) <div class="attribution"> to <footer> ( Used footer tag for footer design)

  I hope you find this helpful

  1
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted over 2 years ago

  Congratulations 👏👏🎉🎉🎉 to complete your challenges. I suggest you in your card design to give border-radius: 13px. I hope you find this helpful

  1
• 

  Jérémy•1,020

  @jrmydixPosted over 2 years ago

  Hi! Well done!

  If this can help, I have a suggestion:

  • Fix accessibility issues: to do so, change your <h3> to <h1>. For the landmark issue, change <div class="container"> to <main> and change your <div class="attribution"> to <footer>

  If you have any questions feel free to ask, I'll try my best to help and answer! 🙂

  1
• 

  kappa•180

  @kappa20Posted over 2 years ago

  Thank you <3

  0