Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Profile Card

lucielub 90

@LucieLuB

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@juan-mentor

Posted

  • Hola Lucielub tengo algunas reflexiones que ofrecerte, si me lo permites:

    • La estructura de tu código es prácticamente perfecta. La única duda es si deberías haber utilizado la etiqueta <nav class ="LinksContainer">...</nav> en vez del <div class= "LinksContainer">...</div>.
  • Gracias a tu código he aprendido a utilizar el atributo role y aria-label de forma correcta.

  • Además es escalable y muy legible, con clases muy descriptivas que no hace falta ningún comentario o explicación de que hace cada elemento.

  • Hay algo que siempre me ha parecido algo contradictorio y es el hecho de resetear estilos, para luego poner margins y paddings a los elementos. No es una crítica, más bien es una reflexión que me hago cuando lo veo plasmado en un código.

          * {
               margin: 0;
               padding: 0;
         }
         .
         .
         .
         element1 {
                 margin-button: 1rem;
         }
        
           element2 {
                 padding: 2rem;
         }
        
     - La verdad es que no puede estar mejor hecho, es sencillo, legible y reutilizable.

###Enhorabuena, buen trabajo.

Marked as helpful

0

lucielub 90

@LucieLuB

Posted

Hola @juan-mentor,

Gracias por tu comentario. Tienes razon, podria haber usado la etiqueta <nav> en vez de un simple <div> ya que semanticamente se puede considerar como un menu de navegation :)

Es una reflexión interesante y válida. La razón principal de usar un 'reset' o 'normalize' es para establecer un punto de partida consistente entre todos los navegadores, ya que cada uno aplica estilos predeterminados diferentes. Al resetear, eliminamos esos valores por defecto para que no interfieran, y luego podemos aplicar los márgenes y paddings de manera controlada y consistente, según las necesidades de diseño. Es más una cuestión de controlar los estilos de forma explícita, en lugar de confiar en los valores predeterminados que pueden variar. Pero sí, parece un poco paradójico a primera vista, ¡hasta que te sumerges en los detalles de compatibilidad entre navegadores!

Muchas gracias de nuevo :)

0

@juan-mentor

Posted

@LucieLuB

Hola LucieLub:

 Gracias por tu explicación sobre el reseteo de los estilos, es que 
 me llamaba mucho la atención el hecho de que en muchos 
 códigos que veía, lo hacían. No sabía el porque ( sobre todo porque únicamente veo mi proyecto en un solo navegador).

A partir de ahora lo visualizaré en varios (lo veía en firefox) para ver esas diferencias. Quizás por eso lo veía ilógico. Perdona por mi ignorancia. A partir de ahora, lo haré como tú.

Muchas gracias por tu comentario.... seguiré aprendiendo.

0
lucielub 90

@LucieLuB

Posted

@juan-mentor un placer, todos estamos aquí para seguir creciendo disfrutando :)

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord