Social Links Using HTML & CSS

Community feedback

• 

  P

  Darkstar•1,100

  @DarkstarXDDPosted 4 months ago

  • You can't use a button inside the <a>. They are two different elements used for two different purposes. <button> is for toggling content like an accordion or submitting a form. <a> is for navigation. Chose the one that matches your need. In this case it's an <a>. Because clicking on those links will navigate to another webpage. You can style the <a> to look like a button.
  • Try to write better alt text. read

  Marked as helpful

  1

  ---

  

  Vinny_A•50

  @Eye0001Posted 4 months ago

  Thank you, I fixed it.

  0
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 4 months ago

  Hey there!

  Congrats on finishing the challenge! ✅

  Your solution looks awesome!

  📌 It's a good idea to use semantic HTML elements like <ul> and <li> for lists. This makes your code more accessible, maintainable, and meaningful.

  Here's an example of how you can refactor your code:

  After Refactoring

  <ul class="list-container">
      <li><a href="#">Github</a></li>
      <li><a href="#">Frontend Mentor</a></li>
      <li><a href="#">LinkedIn</a></li>
      ...
  </ul>
  

  Using <ul> and <li> makes your content structure clearer, which is better for screen readers and search engines. Plus, it follows best practices for HTML.

  Hope this helps!

  Keep up the great work!

  Marked as helpful

  1

  ---

  

  Vinny_A•50

  @Eye0001Posted 4 months ago

  Thanks for the feedback, I will keep this in mind.

  0