4GeeksAcademy / form-cesar

0 stars 0 forks source link

Comentarios #1

Open AlvaroMartinFernandez opened 1 year ago

AlvaroMartinFernandez commented 1 year ago

Buenas Cesar, en la version mvl se ve mal, es por que estas usando estilos en css y con styles.css y estas poniendo un width:50% eso hace que en la version mvl se te vea asi. image

Te recomiendo que utilices bootstrap en toda la practica tienes propiedades para dar anchura y puedes utilizar la sm, md, lg y las otras propiedasdes de la pantalla para que se vea bien el responsive.

Por otro lado como pequeña anotacion te indico que es importante que visualices y tengas como prioridad la version mvl ya que la mayoria del trafico de internet actualmente vienen de mvl

cesarsan86 commented 1 year ago

Hola Álvaro, sólo me queda una duda respecto al background-color gris de la parte de los iconos, aún poniéndole col-sm-12, no engloba a todos los iconos el background-color, y he probado de todo y no sé cómo solucionarlo. Como me recomendaste, he utilizado Bootstrap en toda la práctica, salvo en el color de fondo del "Some fields are missing", ya que ese color no está en Bootstrap.

AlvaroMartinFernandez commented 1 year ago

Buenas Cesar la practica ya esta mucho mejor y se puede aprobar te dejo como solucionar el tema de las tarjetas.

Es por el tamaño de las col image

Si ves el col 6 y lo cambias por un col12 en moviles lo solucionas: adjunto imagenen: image

Por otro lado para tablets que tienes puesto el col-6 deberia ser un col-8 "date cuenta que si tienes un div col6y luego tienes un hijo el tamaño de se hijo se divide tambien en 12 columnas. image

Si no lo tienes muy claro lo podemos ver en una mentoria me escribes por slack para acordar un dia. La practica la podemos aprobar, pero importante que entiendas lo que te explique de los col