Skip to content
OverviewSolutions
2
Comments
2
José Corrales
@JcCorrales94

All comments

  • José Corrales•40
    @JcCorrales94Posted almost 2 years ago

    En plano general está muy bien. Sugerencias a mejorar, no uses todo el rato etiquetas "div" y procura usar las correspondientes, por ejemplo, observé que no tienes ningún "header", más bien pusiste un "main" a la cabecera. Luego a lo que realmente sería el "main" lo llenaste de "div". Te falta mejorar un poco el nombrado de las etiquetas. También te recomiendo que empieces a practicar una metodología BEM para el nombrado de las etiquetas, eso te ayudará mucho luego en tu código CSS.

    Por otra parte, tu CSS lo he visto bastante bien. Por ser un poco quisquilloso, te aconsejaría que no mezclar tanto %, px, vh, em, rem, etc. Sino te enfoques en 1 o 2tipos de medidas. En lo personal uso normalmente % y rem solamente y uno una regla global que te permite que 1 rem sea igual a 10px. La forma es la siguiente por si la quieres probar. html{font-size:62.5%;}. Coloca lo al principio del CSS con tus globales y ya cada vez que uses rem funcionará como te explique.

    Respecto al Responsive, te ha faltado hacer el navegador Responsive. Eso se logra con una función básica de JavaScript que el mismo proyecto te trae.

    Por lo demás, sigue así ^^ vas muy bien.

    0
  • José Corrales•40
    @JcCorrales94Posted almost 2 years ago

    El compañero @Bezies _ tiene razón y si me lo permitís te añadiré una sugerencia extra, para evitar que la imágenes al verse desde un monitor de mayor resolución. En la parte de tu CSS con los "Globales" añádele una etiqueta img{max-width:100%}, de esta forma siempre ocupará el 100% de su contenedor y no se verá cortada. Por lo demás, muy buen trabajo ^^

    0