Design comparison
Solution retrospective
Cualquier comentario para construir se agredece ... :)
Community feedback
- @simonemandolaPosted about 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 helpful0@Sebastian-1406Posted about 3 years ago@simonemandola gracias los tomare en cuenta ... :)
1 -
- @Dike-CodePosted about 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 helpful0@Sebastian-1406Posted about 3 years ago@Dike-Code tome tu recomendacion gracias ... :)
0
Please log in to post a comment
Log in with GitHubJoin 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