Maciel-F / mercado

0 stars 0 forks source link

peer review #1

Open brian25cruz2006 opened 10 months ago

brian25cruz2006 commented 10 months ago

1- En el buscador no se puede escribir y no dirige a ningún lado. 2- Faltaría un poco de estilo en header. 3- En la parte del login y register cuando se envian los datos no se dirige a la home. Despues todo lo demas esta muy bien tanto en mobile como desktop, me gusto.

DrakoZE commented 10 months ago

Header: En el header falta el site osea la imagen que esta arriba de los articulos. En la parte de arriba del header los elementos estan muy separados y tambien estan algo chicos. El padding tambien tiene que reducirse ya que estan muy alejados de los bordes. El texto de los enlaces del header son muy chicos y no se notan tanto. Por ultimo el buscador tiene que ser un formulario en el que puedas escribir y que te rediriga a la home, ademas en modo "mobile" el buscador se rompe un poco

Main: Falta agregar un tipo de transicion al nombre del producto como puede ser cambiar su opacidad cuando lo estes apuntando. La animacion del camion es un poco lenta eso habria que cambiar

Footer: En el footer los iconos no tienen un tamaño de 2em ademas no tienen una animacion al pasar el cursor por estos, ademas el margin-bottom de los iconos debe de ser de 10px. El footer tampoco es flexible es dispositivos superiores a "mobile" y donde dice eso del "copyright" no esta centrado

Login y Register: En estas rutas falta agregar el footer y que el header sea flexible y que aparezcan los enlaces o el "menu-hamburguesa" ademas de poner una transicion en el boton

Y eso seria todo pero buen proyecto xd 👍