social-links-profile-main

Community feedback

• 

  jkaps9•140

  @jkaps9Posted 4 months ago

  HTML is missing the main tag which is standard in HTML5.

  Your screenshot doesn't seem to match your final solution

  Marked as helpful

  1

  ---

  

  Петро•50

  @stadnik-petroPosted 4 months ago

  @jkaps9 Thank you for your thorough comment!

  0
• 

  P

  AKdeBerg•160

  @AKdeBergPosted 4 months ago

  The code looks great. In my code, I used div for the links. Here you used nav tag which is more appropriate, I think.

  Marked as helpful

  1

  ---

  

  Петро•50

  @stadnik-petroPosted 4 months ago

  @AKdeBerg Thank you for commenting on my work, I think you were right to use div instead of nav, because this is a card and not a website, in the navigation bar of a website you should use nav, but now I look and think that in the card my decision was wrong.

  1

  

  P

  AKdeBerg•160

  @AKdeBergPosted 4 months ago

  @stadnik-petro I see! that's a really helpful tips. thanks

  0