search

MelinaLeonardi / Anatomia_PaginaWeb

Desde clase17- modulo4 de DH, extructuramos nuestro proyecto de pagina web, complejizando en cada clase el nivel de codigo.
0 stars 0 forks source link

Peer Review #1

Open martinezmariana opened 1 year ago

martinezmariana commented 1 year ago

✔ 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.

  1. podria sacarle el z-index o la posicion del header lo demas esta genial.
MelinaLeonardi commented 1 year ago

Mari la transición e l camión es a partir de la vista tablet o desktop no me acuerdo bien. Tenía hecha la transición. Tuve un quilombito xq quise restaurar a versión anterior xq me puse a toquetear todo el código y después me arrepentí.

El lun., 9 de enero de 2023 11:45, marianamartinez @.***> escribió:

✔ 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.

  1. podria sacarle el z-index o la posicion del header lo demas esta genial.

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

martinezmariana commented 1 year ago

Hola meli aah perdón no lo vi x eso. A hora después lo acomodo.Enviado desde mi smartphone Samsung Galaxy. -------- Mensaje original --------De: Melina Leonardi @.> Fecha: 10/1/23 10:38 (GMT-03:00) A: MelinaLeonardi/Anatomia_PaginaWeb @.> CC: marianamartinez @.>, Author @.> Asunto: Re: [MelinaLeonardi/Anatomia_PaginaWeb] Peer Review (Issue #1)

Mari la transición e l camión es a partir de la vista tablet o desktop no

me acuerdo bien. Tenía hecha la transición. Tuve un quilombito xq quise

restaurar a versión anterior xq me puse a toquetear todo el código y

después me arrepentí.

El lun., 9 de enero de 2023 11:45, marianamartinez @.***>

escribió:

✔ 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.

  1. podria sacarle el z-index o la posicion del header lo demas esta

genial.

—

Reply to this email directly, view it on GitHub

https://github.com/MelinaLeonardi/Anatomia_PaginaWeb/issues/1, or

unsubscribe

https://github.com/notifications/unsubscribe-auth/A2AO2MS4ETBQLDAYRV343ILWRQQCNANCNFSM6AAAAAATVQOYYE

.

You are receiving this because you are subscribed to this thread.Message

ID: @.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

[

{

@.***": "http://schema.org",

@.***": "EmailMessage",

"potentialAction": {

@.***": "ViewAction",

"target": "https://github.com/MelinaLeonardi/Anatomia_PaginaWeb/issues/1#issuecomment-1377299968",

"url": "https://github.com/MelinaLeonardi/Anatomia_PaginaWeb/issues/1#issuecomment-1377299968",

"name": "View Issue"

},

"description": "View this Issue on GitHub",

"publisher": {

@.***": "Organization",

"name": "GitHub",

"url": "https://github.com"

}

}

]

MelinaLeonardi commented 1 year ago

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)