Profile card component, CSS Flex

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should also take notes of the following…

  • Clean the whitespaces in the code
  • Remove unnecessary code to keep it clean
  • Include description with the alt in image tags. Like this line <img src="./images/image-victor.jpg" alt="">
  • Adjust the border size into border-top: 0.01rem solid var(--dark-gray); in hr rule set

  Above all, The design looks good. Keep it up! Hope this is helpful!

  Marked as helpful

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Rony Good Job on challenge

  These below mentioned tricks will help you remove any Accessibility Issues

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

  -> you can't use same id more then once, in that case use class,

  -> 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
• 

  Hexer•3,660

  @EmmanuelHexerPosted almost 3 years ago

  Good job overall man. Nice work

  0

  ---

  

  Rony•420

  @romzcPosted almost 3 years ago

  @Phalcin thanks

  0