VCamila / dwf-desafio-mod4

0 stars 0 forks source link

Review #2 - Detalles en la page y formulario. #2

Open FacundoEG opened 2 years ago

FacundoEG commented 2 years ago

Hola Cami! estoy viendo tu desafío y encuentro algunas cositas que se rompen en la page:

1.En servicios te podría sugerir que uses flex-wrap como en el portfolio ya que las cards se empiezan a amontonar en resoluciones bajas: Screenshot_20220308_131902

2.Luego el icono superior de la izquierda no te lleva a ningun lado, fijate el tema del href que le estas dando un elemento con id que no existe al parecer. Esto revisalo porque si no el usuario no puede volver a la pagina principal. Screenshot_20220308_131920

3.En la page principal, la sección "section" se queda fuera del viewport cuando vas bajando de resolucion: Screenshot_20220308_132008

Esto con sacarle el width al parrafo se arregla! Screenshot_20220308_132407

4.Otra cosita, fijate que el form se queda un poco estatico en ciertas resoluciones, te recomendaria que el padding del costado del contenedor lo pongas en porcentajes asi es mas dinamico: Screenshot_20220308_132504

5.Y lo que si te pediría que revises el tema del formulario porque estoy viendo tu codigo y no hace lo que debería con los alerts, chequeate que cuando envias el formulario se vuelve a recargar la pagina y no debería suceder eso: Screenshot_20220308_132614

Son muchas cositas muy pequeñas, mas que nada te pediría que arregles el tema del form que recarga la pagina y lo de las cards de servicios, lo demas es mas sugerencia para mejorar lo visual!

Cualquier cosita acordate que podes enviar tus dudas por discord y te ayudamos! Saludos!

VCamila commented 2 years ago

Hola Facundo, gracias por tu revisión. Ahí me estuve fijando todos los puntos en los que me hiciste alguna observación o recomendación y ya pude resolver prácticamente todos, menos el punto 2 y el 5 sobre el formulario. Pero es en el punto 5 en el que estoy teniendo problemas para entender a que te referís con tu revisión, no entiendo a que te referís con que el form recarga la página o que no hace lo que debería con los alerts. Cuando completo el formulario, se envía el mensaje porque me llega a mi email, me aparece la alerta y le doy a aceptar. Vos te referís a que luego de todo esto, el formulario sigue completado con los datos que yo agregué? Disculpa si es una pavada pero no entiendo bien a que te referís con que "recarga la pagina". Gracias

FacundoEG commented 2 years ago

Camí me refiero a que cuando pruebe usar el form en el momento que hice la revisión, la página hizo reload() y se vuelve a recargar sin darme ningún alert o nada.(Recien lo probé desde el celu y hace lo mismo). Ahora lo volveré a chequear desde la compu y cualquier cosita te aviso si?

VCamila commented 2 years ago

Ok, Facu. Cualquier cosa me decís. Estoy en eso fijándome pero a mi no me hace reload(), gracias.

FacundoEG commented 2 years ago

Cami ahi lo probe y solo hace el reload en el formulario de la página principal!

El formulario de la página contacto anda de 10, error mio por no aclarartelo.

Te recomendaria de revisar la logica que esta ahi o si la pagina principal llama a la función del formulario si?

VCamila commented 2 years ago

Ah Facundo, que boluda no me di cuenta. Tenés razón, ahora lo modifico y luego lo actualizo. Gracias

FacundoEG commented 2 years ago

Gracias a vos y perdon por la poca precisión jaja cualquier cosita avisa!

VCamila commented 2 years ago

Hola @FacundoEG, recién actualicé la página con todos los cambios que me dijiste que haga. Quisiera que te fijes si está todo bien antes de subir el desafío en la página de apx. Gracias, saludos.

FacundoEG commented 2 years ago

Hola Cami! lo unico que encontre es esto en la pagina de portfolio, que se footer se eleva de golpe hacia arriba:

image

Me parece que es por el alto que le estas dando fijo a la sección de donde estan las cards, de todas formas es algo que se nota si lo rebuscas mucho, yo creo que si arreglas eso ya esta totalmente aprobado el desafío! de todas formas volvelo a enviar porque te tiene que revisar otro tutor y dar el aprobado si.

Te mando un saludo y cualquier cosa me consultas si?

VCamila commented 2 years ago

Hola Facu! Ah ok, bueno modificaré eso y luego lo mando. Muchas gracias por tu ayuda, Facu