Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Peer Review #1

Open
martinezmariana opened this issue Jan 9, 2023 · 3 comments
Open

Peer Review #1

martinezmariana opened this issue Jan 9, 2023 · 3 comments

Comments

@martinezmariana
Copy link

✔ 1. cumple con todo.
✔ 2. el footer esta centrado y los icons tambien. cumple con todo.
✔ 3. se adapta a todas las pantallas.
❌ 3. lo que queda fijo es el header y inconoda la vista en mobile y tablets.
✔ 4. cuenta con el formulario de registro. olvido dar estilos.
✔ 5. cuenta con formulario de login. olvido dar estilos.
✔ 6. si muestra error.
❌ 7. no es un formulario .
✔ 8. usa animaciones y transiciónes en css.
❌ 8. falto dale animacion al camion.
✔ 9. muy bien con todo. resuelto todos los puntos.
10. podria sacarle el z-index o la posicion del header lo demas esta genial.

@MelinaLeonardi
Copy link
Owner

MelinaLeonardi commented Jan 10, 2023 via email

@martinezmariana
Copy link
Author

martinezmariana commented Jan 10, 2023 via email

@MelinaLeonardi
Copy link
Owner

MelinaLeonardi commented Jan 11, 2023

#3. HEADER REACOMODADO, SIN FIXED (NO ESTA FIJO EN LA PANTALLA) > SIN Z- INDEX

❌ 7. no es un formulario .PORQUE NO SE CONSIDERA FORMULARIO SI ESTA ENCERRADO ENTRE ETIQUETA FORM?

#8. EL CAMION TIENE TRANSICION AL IGUAL QUE LA DESCRIPCION A PARTIR DE VISTA TABLET 768 PX APROXX FIJATE EN RENDER QUE A PARTIR DE ESA VISTA SE ACTIVA EL BOX SIZING Y LA VISUALIZACION DE LA DESCRIPCION AL POSICIONARTE EN EL ARTICLE (PRODUCT :HOVER )Y AL POSICIONARTE )EN EL CAMION DE ENVIO, ESTE GIRA 360° (TRANSITION :HOVER)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants