analuz20unla / laptoplenovo

0 stars 0 forks source link

Correcciones #1

Open NicoPedreira opened 3 weeks ago

NicoPedreira commented 3 weeks ago

Hola. Viene muy bien el tp por ahora. Te doy una devolución basada en las dudas que comentaste en el Readme.

Los estilos que aplicaste inline deberían estar en la hoja de estilos. Una forma, por ponerte un ejemplo, de darle un font weight determinado a las etiquetas span sería haciéndolo directamente como: span{ font-weight: x; } En el caso de que quieras darle diferentes pesos a la fuente, podes diferenciarlos utilizando una class.

Otros temas como el acomodado de ciertas secciones y los espacios entre ellas (vi que usaste estilos inline para eso también) lo vas a poder resolver luego de que veamos flexbox en la clase de hoy. Por el momento está muy bien encarado el trabajo.

Por último, vi que usaste algunas etiquetas hgroup, que no son semánticamente correctas para esa aplicación. Te recomiendo reemplazarlas por otra, por ejemplo, un article

analuz20unla commented 3 weeks ago

Gracias por la devolución Nico ¿me podrás decir que etiquetas no usé corretamente que pueden reemplazarse por article u otra? Para tener idea de cual es el error en específico. Mil gracias!

NicoPedreira commented 3 weeks ago

Hola, perdón, ahí lo corregí