Social Link Profile Main

Community feedback

• 

  Daniel 🛸•44,270

  @danielmrz-devPosted 9 months ago

  Hello @abandonedwaffle!

  Your solution looks excelent!

  I have just one suggestion:

  📌 To improve semantic clarity, opt for <h1> over <h2> for your main title.

  It's more than just text size — it's about structuring your content effectively:

  • <h1> to <h6> are used to define HTML headings, with <h1> being the most significant.
  • Stick to one <h1> per page for the main title, and maintain the titles hierarchy with <h1>, <h2> <h3>, and so on.

  While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.

  Hope this suggestion proves helpful! Keep up the great work!

  Marked as helpful

  1

  ---

  

  Ritesh Diwan•50

  @abandonedwafflePosted 9 months ago

  Thanks @danielmrz-dev for the suggestion. I have made changes according to it. I will keep it in mind for doing future projects. This was helpful.

  1
• 

  Madhu U•170

  @Madhu-UPosted 9 months ago

  Good Job!

  • You should change all the buttons for anchor tags. Buttons and links serve different purposes. These are links. They'll navigate to another page when you click on them.
  • I would recommend you use "rem" instead of "px" for font size for accessibility reasons.

  Marked as helpful

  1

  ---

  

  Ritesh Diwan•50

  @abandonedwafflePosted 9 months ago

  Thanks @Madhu-U for the feedback. I will be switching to only using rem for fonts. I have also made the necessary changes you pointed out. Thanks again.

  0