search

havikhavik / apx-desafio-m4

0 stars 0 forks source link

Issue #2

Closed MatiToledo closed 3 years ago

MatiToledo commented 3 years ago

Hola Lucas !! Estuve viendo la page y esta excelente, todo increíble, solo note 2 cositas que habría que arreglar para terminar de pulir la page

  1. En mobile, en la "hero-section" la imagen se duplica en vez de verse entera, habria que ponerle algo para que la imagen se expanda a todo el contenedor, podría ser algo como un background-size: cover;
  2. Hay un problema tanto en portfolio como en servicios que en cierto momento la page crashea, habria que ver de solucionar eso con las media queries l1

Pero todo el resto espectacular !!!

havikhavik commented 3 years ago

Hola Matías, gracias por mirar y probar mi Portfolio!

En cuanto a los problemas de layout que mencionas, 1) Implementando la propiedad background-size: cover; en el .hero-section la imagen ahora ocupa todo el ancho de pantalla 2) ocurría porque el ultimo elemento rompía el ancho en pantalla, lo pude solucionar implementando la propiedad grid-template-columns: repeat(auto-fit, minmax(320px, 1fr)); Lo que hace esta propiedad es definir de forma automática la cantidad de columnas dependiendo el tamaño de la pantalla.

Gracias por chequear el trabajo y subo capturas con el diseño funcionando correctamente!

fixed fixed3 fixed2 fixed4