Social Link Page

Community feedback

• 

  P

  Raphael Perfi•290

  @perfidevPosted 10 months ago

  Very well, my friend! It looks great! Here are some suggestions:

  • I would add "href" to the <a> tags because they are links and for best practices.
  • I would include a <nav> tag with a <ul> list and place the <a> links inside <li> tags, as it adds semantics since you are navigating through the links.
  • I would set the body as display: flex to center your <div>.

  <div class="box-botoes">
      <nav>
          <ul>
              <li><a href="https://github.com">Github</a></li>
              <li><a href="https://frontendmentor.io">Frontend Mentor</a></li>
              <li><a href="https://linkedin.com">LinkedIn</a></li>
              <li><a href="https://twitter.com">Twitter</a></li>
              <li><a href="https://instagram.com">Instagram</a></li>
          </ul>
      </nav>
  </div>
  

  Marked as helpful

  0

  ---

  

  Samuka Gomes Dev•40

  @samukagomesdvPosted 10 months ago

  Thanks for the sugestion! @raphaelperfi

  1
• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 10 months ago

  Fala @samukagomesdv!

  Seu projeto está excelente!

  Tenho uma sugestão:

  📌 Use <main> pro conteúdo principal ao invés de uma <div>.

  Tags como <div> e <span> são exemplos típicos de elementos HTML não-semânticos. Elas servem apenas como containers para o conteúdo, mas não indicam qual o tipo desse conteúdo e nem o papel que ele desempenha na página.

  Essas mudanças de tag geram pouco ou nenhum impacto visual mas tornam o seu código HTML mais semântico e melhoram a otimização SEO e acessibilidade do projeto.

  Espero que ajude!

  Fora isso, ótimo trabalho!

  0

  ---

  

  Samuka Gomes Dev•40

  @samukagomesdvPosted 10 months ago

  Olá, @danielmrz-dev!

  Bacana, obrigado pela dica. Estou aprendendo e aos pouquinhos melhorando esses detalhes!

  0