Mejoras evaluación intermedia

pixelmary commented 4 years ago

Buen trabajo en general Laura! Te indico algunas mejoras para rematar el ejercicio.

HTML

Falta el idioma en la etiqueta HTML.
El menú debe ir maquetado en una lista UL y LIs, un menú es un listado de enlaces

Es mejor mover la tipografía antes de tu CSS principal
Mejora la jerarquía agregando estilos más comunes a la etiqueta del cuerpo como tamaño de fuente, familia de fuente, peso de fuente, color, color de fondo

Hay algunos márgenes extraños en la cabecera
En lugar de agregar espacio adicional en la parte inferior del encabezado, puedes hacer que los enlaces del menú sean más altos, eso haría que el encabezado sea lo suficientemente alto y también haría que el enlace sea más fácil de tocar en dispositivos móviles
Faltan por solucionar los espacios en la cabecera, deberías incorporar aire a los lados y separar los items de menú.

Recuerda que debes especificar el tamaño de tu etiqueta H1
Es recomendable usar REM en el tamaño de fuente e indicar el tamaño base en la etiqueta HTML
Da color en el HOVER a los enlaces

Te recomiendo que corrijas todos los detalles, te vendrá genial para asimilarlos y no arrastrarlos a próximas evaluaciones.

Avísame si tienes dudas ;)

lauraherrero commented 4 years ago

Hola María,

He vuelto a subir la evaluación con las correcciones tanto en el index como en el css.

[AÑADIDO ] Falta el idioma en la etiqueta HTML.
[ CORREGIDO] El menú debe ir maquetado en una lista UL y LIs, un menú es un listado de enlaces
[CORREGIDO ] Es mejor mover la tipografía antes de tu CSS principal
[CORREGIDO ] Mejora la jerarquía agregando estilos más comunes a la etiqueta del cuerpo como tamaño de fuente, familia de fuente, peso de fuente, color, color de fondo
[CORREGIDO ] Hay algunos márgenes extraños en la cabecera
[ CORREGIDO] En lugar de agregar espacio adicional en la parte inferior del encabezado, puedes hacer que los enlaces del menú sean más altos, eso haría que el encabezado sea lo suficientemente alto y también haría que el enlace sea más fácil de tocar en dispositivos móviles
[ CORREGIDO] Faltan por solucionar los espacios en la cabecera, deberías incorporar aire a los lados y separar los items de menú.
[AÑADIDO ] Intenta usar flex en la cabecera
[YA ESTABA ] Recuerda que debes especificar el tamaño de tu etiqueta H1
[AÑADIDO ] Es recomendable usar REM en el tamaño de fuente e indicar el tamaño base en la etiqueta HTML
[AÑADIDO] Da color en el HOVER a los enlaces

Muchas gracias!

pixelmary commented 4 years ago

Muy bien @lauraherrero , ahora sí que lo has bordado!. Cierro tu ISSUE