Responsive Design, sass , flexbox y grid

Community feedback

• 

  Eklemis Santo Ndun•620

  @eklemisPosted almost 3 years ago

  Completed nicely!

  Marked as helpful

  0
• 

  Simone Mandola•100

  @simonemandolaPosted almost 3 years ago

  Hola! Muy buen trabajo!

  Si puedo te aconsejaría de:

  • envolver el logo en una etiqueta "a"

  • los elementos del menú principal llenen de un 100% del height del menú

  • (en este caso) que todos lo botones "Request Invite" sean enlaces por que son "CTA" y seguramente en un caso real te llevarían a la página de contacto o similares

  • envolver los títulos de los artículos en una etiqueta "a" para que sean pulsables

  • trabajar con medidas "rem"

  Además te aconsejaría de fijar un "max-width" para todo el contenido del sitio web y cuidado con la imagen de fondo del "header"...

  Espero que estos consejos te sirvan para futuros trabajos. 😀😀😀

  Marked as helpful

  0

  ---

  

  Sebastian•770

  @Sebastian-1406Posted almost 3 years ago

  @simonemandola gracias los tomare en cuenta ... :)

  1
• 

  Dike Obinna Clinton•200

  @Dike-CodePosted almost 3 years ago

  All is good from the starting point and quite responsive but I think you should work on the "footer" by adding a margin-top on the social icon's container just to space it a little from the logo... Great Job!!

  Marked as helpful

  0

  ---

  

  Sebastian•770

  @Sebastian-1406Posted almost 3 years ago

  @Dike-Code tome tu recomendacion gracias ... :)

  0