CSS Flexbox

Community feedback

• 

  Bhremada Fevreano•1,395

  @boedegoatPosted almost 3 years ago

  Great work AditNovadianto. You've done fantastic job

  In order to fix your accessibility issue, you can add <main> tag to be the wrapper of your entire component.

  Other than that, stay code and happy coding

  Marked as helpful

  1

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @boedegoat thnakyou for your advice next i will practice more about <main> tag

  0
• 

  Mohammad Ebrahimi•250

  @moheb2000Posted almost 3 years ago

  Awesome 👏👏👏

  1- For 'Why Us' section it's better to use lists in HTML because it actually is a list but if you don't want to use that you need to put them in different lines.

  2- don't use flex method for centering element in mobile view because it's cause overflow issues. You fixed overflow in vertical direction but in horizontal direction some texts aren't readable.

  Great work. Good lock ❤️❤️❤️

  Marked as helpful

  0

  ---

  

  AditNovadianto•1,040

  @AditNovadiantoPosted almost 3 years ago

  @moheb2000 thankyouu for your advice!!!!

  0