Social link profile

Community feedback

• 

  P

  Justin Connell•720

  @justinconnellPosted 10 months ago

  Hi @Carolina-lab1,

  Your submission looks great on the design comparison!

  I reviewed your solution and it looks good on desktop devices - but, when trying to view it on other resolutions I noticed that the solution does not support tablet/mobile screen resolutions.

  I then looked at the code and have some recommendations I hope you will find helpful.

  • try structuring your HTML using more semantic elements.
  • name classes after the thing they represent - for example instead of using classes that describe a navigation bar, use class names that describe the parts of the social links profile card parts - profile, name, location, about, social links are the parts that make up the component, so by using those class names makes the code easier to understand and provides clear context on what each part of HTML and CSS represents.

  Great going so far, keep learning, applying and moving forward!

  0