Eklemis Santo Ndun• 620
@eklemis
Posted
Completed nicely!
Marked as helpful
0
Design comparison
Solution retrospective
Cualquier comentario para construir se agredece ... :)
Community feedback
Simone Mandola• 100
@simonemandola
Posted
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-1406
Posted
@simonemandola gracias los tomare en cuenta ... :)
1
Dike Obinna Clinton• 200
@Dike-Code
Posted
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-1406
Posted
@Dike-Code tome tu recomendacion gracias ... :)
0
Please log in to post a comment
Log in with GitHub
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