7carlinger
commented
4 years ago Gracias Fede, recien lo veo hoy 1/10. Perdon. Ya se lo pasé a mis compañeras
Open fedm4 opened 4 years ago
7carlinger
commented
4 years ago Gracias Fede, recien lo veo hoy 1/10. Perdon. Ya se lo pasé a mis compañeras
El contenedor del header tiene un ancho maximo, cuando por lo visto deberia ocupar el 100% asi de un lado ponemos el titulo y del otro el menu
El hero no tiene ninguno de sus elementos bien posicionados. La imagen bien podria ser un background en lugar de un img, y centrar el texto y el boton debajo.
El div class container dentro del hero afecta directamente la seccion about us, moviendola hacia la izquierda. Esto es porque esta hecho con float en lugar de flex. Deberian usar flex para esto.
El about us esta perfecto pero le agregaria un "margin: 0 auto" para que se centre su contenido, una vez arreglado el punto anterior.
Services esta muy bien armado.
Why choose us faltaron estilos de la seccion, los colores de la fuente deberian ser blancos los weight de las fuentes no estan del todo correctos en el subtitulo.
Our portfolio hay que rehacerlo. Por empezar deberia ocupar el 100% del ancho de la pantalla, pero tiene un maximo definido que le impide pasar a mostrar 3 o 4 imagenes por fila. Ademas los colores deberian revisarse, los botones y las fuentes tambien
En our portfolio, especificamente las imagenes necesitan un "max-width: 100%" para que no se pasen de su padre, o bien, ponerle al padre de las imagenes un "overflow: hidden" para que no quede nada fuera de cada card.
El footer esta muy muy bien.