profile-links-profile-main

Community feedback

• 

  Harsh Kumar Dwivedi•740

  @Harsh-Kumar-DwivediPosted 8 months ago

  Hi @HansKevin,

  Nice Solution !

  I would suggest following:-

  1.Prefer using <main> element instead of <div class="container">.

  2.Consider replacing <div class="attribution"> with <footer> element.

  3.Providing fallback fonts are useful.

  4.In this project, I think it is not useful to select container class as a part of CSS rule selector every time.

  5.Understanding CSS rule selector specificity could be helpful.

  These will enhance your code, improve accessibility and make your markup more semantic.

  Hope these suggestions are helpful !

  Stay Happy & Well !

  Happy Learning !

  Marked as helpful

  0

  ---

  

  HansKevin•40

  @HansKevinPosted 8 months ago

  @Harsh-Kumar-Dwivedi thanks

  1

  

  Harsh Kumar Dwivedi•740

  @Harsh-Kumar-DwivediPosted 8 months ago

  @HansKevin

  Welcome !

  Keep up the good work !

  0