CSS Grid and CSS Flexbox

Community feedback

• 

  Bhremada Fevreano•1,395

  @boedegoatPosted almost 3 years ago

  Great work AditNovadianto !

  Don't forget to add <main> tag to be the container of your entire app. That will fix those accessibility issue

  Other than that, keep it up and happy coding

  Marked as helpful

  1

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @boedegoat thankyouu so much bhremada!!! i will fix that accessibility issue for the best project

  0
• Account deleted

  Hi there 👋

  Congratulate on finishing your project 🎉. You did a great job 💡

  I give some suggestions to help you take your project design to the next level 📈😉

  • Change the background color to EDF2F8
  • And change the background of the card about Jeanette Harmon and Kira Whittle to #ffffff
  • Make the box shadows a bit lighter

  Happy coding ☕

  Maqsud

  Marked as helpful

  0

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @maqsudtolipov thank youu for your suggestion maqsud tolipov!!!

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! AditNovadianto Good Job on challenge

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body <main class="container"></main>

  -> Learn more on accessibility issues

  If this comment helps you then pls mark it as helpful!

  Have a good day and keep coding 👍!

  Marked as helpful

  0

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @Crazimonk thank youu for your advice naveen gumaste!!!

  1
• 

  optimusprime202•1,200

  @optimusprime202Posted almost 3 years ago

  Hey @AditNovadianto, You did a lot of work today.

  1

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @optimusprime202 thank you in advance, because I like working on a project

  0