Social links profile FlexBox

Community feedback

• 

  isaac nabil•220

  @nabil4416Posted 9 months ago

  hi, great job, it was a good idea to use the "flex" regarding the "border-radius" that you used to circle the profile photo, you can use "50%" instead of "9999px", and no need to add a "height" for it.

  Marked as helpful

  0

  ---

  

  Royer Rutti Ramos•200

  @WizardRRRPosted 9 months ago

  @nabil4416 right, I'll change it, thank you

  0
• 

  Bishal Singh Deo🎮•1,440

  @Bishalsnghd07Posted 9 months ago

  Hi, @WizardRRR

  Congrats for completing this challenge🎉

  I have one suggestion for you, might be helpful:

  I would recommend to maintain the semantic ui approach in your html. To improve your semantic ui follow the heading levels like this👉 h1, h2, h3, h4, h5 & h6. After h1 h2 will be there. you have been used directly h3 after h1. To improve your semantic and code quality. Please follow the heading levels in ascending order.

  Hope, this small tip will help you in further project to imrove semantic ui.

  Happy Coding🎮

  0