Nathymmd / proyecto1

Mercado Liebre
0 stars 0 forks source link

Peer review (Alex Cho) #1

Closed allexcho closed 12 months ago

allexcho commented 1 year ago

Hola Nathaly Meier Moreno Como estas Nathaly!! Te comento por las dudas, me cuesta muchisimo expresarme en castellano asi que cualquier cosa me podes hablar aparte si te quedo alguna duda!

1) HEADER

  1. LOGO: Pusiste el tamaño para mobile pero no cambiaste el tamaño para desktop en media queries! con un poco de modificacion a eso quedaria lindo!
  2. BARRA DE BUSQUEDA: Te falto juntar el searchbar con el icono de search y agrandarle un poco mas para cuando vas agrandando y achicando se vea mas lindo!
  3. COMPRA EN CUOTAS Y SIN TARJETA DE CREDITO: Esta bien hecho! solo habra que darle un tamaño para mobile y desktop y va como trompada!
  4. BARRA DE NAVEGACION: Creo que si sacas el width: 42% y le agregas un justify-content: space-around quedaria divino, y para mobile quedo genial!
  5. OFERTA / TIENDAS OFICIALES / VENDER / AYUDA: Esta bien, solo hay que darle espacio en cada cosa para que se vea mejor!
  6. LOGIN: Todas las consignas que pide esta, solo faltaria ordenarlo un poco y para que se vea un poquito mas lindo!
  7. REGISTRO: Como el login, esta todo pero solo faltar acomodarlo para que quede visualmente mas lindo!
  8. MIS COMPRAS: Puede ser que no tengas mis compras?

2) MAIN

  1. BASADO EN TU ULTIMA VISITA: El hover and barbaro, le falta ese toque de shadow, centralizarlos, y agregarle las otras 4 imagenes para OFERTA! Tambien cuando voy achicando para verlo en mobile se rompe en macbook!
  2. OFERTA: Te falto esta parte :(

3) FOOTER

  1. PAGA CON TARJETA O EFECTIVO / ENVIO GRATIS DESDE $2.500 / SEGURIDAD DE PRINCIPIO A FIN / COPYRIGHT: Esta todas las consignas pero para desktop tendria que estar uno alado del otro y ponerlo un poco mas grande para que sea legible! Tambien falto darle ese :hover y darle un color a los iconos para que visualmente quede mas lindo!

Haciendo foco en los estilos del pie de página (Footer):

  1. Te falto centralizarlos!!
  2. El tamaño del icono y letras son iguales! tendrias que agrandar un poco mas a los iconos y darle margen para que no queden tan pegados!

Media queries: Hay partes que se adaptan pero tambien hay partes que no Formulario: Si hay formulario! Login: Si hay login! Mensaje de error formulario: Si muestra error! Searchbar es un formulario: No Animacion/Transicion: Si! en la parte de productos!

FEEDBACK Quedo todo donde tienen que estar! Solo falta darle ese toque para y vas de 10!

Intenta averiguar un poco sobre lo que es el flex, una herramienta genial sin duda lo mejor para estos tipos de trabajos para darle un poco mas de vida!

Nathymmd commented 12 months ago

Alex tus observaciones están abiertas puedes por favor ya cerrarlas. Gracias PS: habia olvidado de cargar el proyecto de nuevo. Entre mis viajes y trabajo. saludos Nathy

El mar, 30 may 2023 a la(s) 20:22, Alex cho @.***) escribió:

Hola Nathaly Meier Moreno Como estas Nathaly!! Te comento por las dudas, me cuesta muchisimo expresarme en castellano asi que cualquier cosa me podes hablar aparte si te quedo alguna duda!

  1. HEADER

  2. LOGO: Pusiste el tamaño para mobile pero no cambiaste el tamaño para desktop en media queries! con un poco de modificacion a eso quedaria lindo!

  3. BARRA DE BUSQUEDA: Te falto juntar el searchbar con el icono de search y agrandarle un poco mas para cuando vas agrandando y achicando se vea mas lindo!

  4. COMPRA EN CUOTAS Y SIN TARJETA DE CREDITO: Esta bien hecho! solo habra que darle un tamaño para mobile y desktop y va como trompada!

  5. BARRA DE NAVEGACION: Creo que si sacas el width: 42% y le agregas un justify-content: space-around quedaria divino, y para mobile quedo genial!

  6. OFERTA / TIENDAS OFICIALES / VENDER / AYUDA: Esta bien, solo hay que darle espacio en cada cosa para que se vea mejor!

  7. LOGIN: Todas las consignas que pide esta, solo faltaria ordenarlo un poco y para que se vea un poquito mas lindo!

  8. REGISTRO: Como el login, esta todo pero solo faltar acomodarlo para que quede visualmente mas lindo!

  9. MIS COMPRAS: Puede ser que no tengas mis compras?

  10. MAIN

  11. BASADO EN TU ULTIMA VISITA: El hover and barbaro, le falta ese toque de shadow, centralizarlos, y agregarle las otras 4 imagenes para OFERTA! Tambien cuando voy achicando para verlo en mobile se rompe en macbook!

  12. OFERTA: Te falto esta parte :(

  13. FOOTER

  14. PAGA CON TARJETA O EFECTIVO / ENVIO GRATIS DESDE $2.500 / SEGURIDAD DE PRINCIPIO A FIN / COPYRIGHT: Esta todas las consignas pero para desktop tendria que estar uno alado del otro y ponerlo un poco mas grande para que sea legible! Tambien falto darle ese :hover y darle un color a los iconos para que visualmente quede mas lindo!

Haciendo foco en los estilos del pie de página (Footer):

  1. Te falto centralizarlos!!
  2. El tamaño del icono y letras son iguales! tendrias que agrandar un poco mas a los iconos y darle margen para que no queden tan pegados!

Media queries: Hay partes que se adaptan pero tambien hay partes que no Formulario: Si hay formulario! Login: Si hay login! Mensaje de error formulario: Si muestra error! Searchbar es un formulario: No Animacion/Transicion: Si! en la parte de productos!

FEEDBACK Quedo todo donde tienen que estar! Solo falta darle ese toque para y vas de 10!

Intenta averiguar un poco sobre lo que es el flex, una herramienta genial sin duda lo mejor para estos tipos de trabajos para darle un poco mas de vida!

— Reply to this email directly, view it on GitHub https://github.com/Nathymmd/proyecto1/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7IEVETK4JH6QCIMNY5FTWTXIY3HXANCNFSM6AAAAAAYUMKE74 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Movil: +41 78 819 56 74 Correo electronico: @.***