JSConfCL / 2024

Página oficial de JSConf Chile 2024
https://jsconf.cl/
3 stars 1 forks source link

geekha/refactor footer #12

Closed geekhadev closed 4 months ago

geekhadev commented 6 months ago

Se envía primera fase de la PR para revisión.

Hay dos temas que no supe resolver en primera instancia:

Si hay ideas les agradezco no soy experto en FrontEnd.

joseglego commented 6 months ago

En general se ve bien

Pero voy a tener que probarlo localmente para poder ver la UI. Mi recomendación personal (que debo agregar como template al repo) es agregar un par de screenshots cuando hayan cambios visuales para así tener una idea.

Respecto a tus comentarios:

  1. Tengo que verlo más en detalle, dejame ver
  2. Dependiendo de cuánto afecte la UI yo sigo esta misma idea. Así que no vería problema si no se afecta tanto. Igual tendría que chequearlo con la diseñadora
geekhadev commented 6 months ago

He agregado unas mejoras para finalizar el footer dejo las captures de como quedó:

Versión desktop:

Screenshot 2024-03-28 at 12 30 12 PM

Versión mobile

Screenshot 2024-03-28 at 12 30 30 PM

Nota: hice un pequeño cambio en el formulario de suscripción: no agregue el label "suscribete" y en versión mobile, organice el botón debajo del input y no al lado

geekhadev commented 6 months ago

Hay que refactorizarlo cuando se integre el Sanity, pero como está creo que cumple para el merge.

geekhadev commented 6 months ago

Se elimina antiguo footer en src/app/page.tsx

joseglego commented 4 months ago

Esto se continuó en el PR: https://github.com/JSConfCL/2024/pull/36

Con @geekhadev como contribuidor interno.