FdelMazo
commented
7 months ago Hola @lopezac !
Primero, gracias por este PR y los otros. Muchas gracias por involucrarte en el proyecto.
En cuanto al PR en sí, espero que no te ofendas, pero no lo quiero incluir. Me gusta como queda el modal en el tercio de abajo del sitio, lo siento menos intrusivo (aunque para ser sincero, re-leyendolo me doy cuenta que quedo muy grande; en algun momento lo tengo que refrasear).
Eso sí, me parece genial la investigación que hiciste del toast component.
PD 1: No crees que la feature de sugerencias tendria que ser separada en su archivo, teniendo tanta logica y para que sea mas facil una transicion a TypeScript?
Recontra! Con el tiempo el archivo fue creciendo mucho y nunca hice el salto de pasarlo para otro lado, pero me parece genial eso. Así también queda reutilizable para otros proyectos, entre otras cosas.
PD 2: Crees que sea necesario aplicar Prettier al proyecto para que quede todo mas ordenado y legible, o preferis el formateo actual del proyecto?
Hiper recontra! Cuando arranque el proyecto no sabía que existian los formateadores, y no es algo que suelo usar en mi día a día entonces nunca hice el paso a uno.
Sea prettier u otro (desconozco), estaría bueno encontrar uno o un set de reglas que modifique lo menor posible el código así no rompe tanto el historial de git, pero mas allá de eso, un formateador que sea lo mas plug and play posible y sin mucho setup requerido es una gran adición al proyecto.
Así se pueden hacer mil cosas (un linter en el CI, por ejemplo) pero siempre quiero intentar evitar sobre-complicar el repo, en particular considerando que es un proyecto medio hobby que no tiene sentido que crezca infinitamente.
Sentite libre de hacer las modificaciones que te copen, y te pido disculpas por no agregar este PR. Si alguna vez preferís abrir un issue antes que un PR para charlar un feature antes de codearlo, también podemos verlo ahí, pero como te parezca a vos.
lopezac
Hola buenas tardes.
Lo principal es que agregue un Center Component y centre el Toast de sugerencias, y el mambo es que el Toast Component tiene un margin de 0.5 rem por defecto y el margin top es ignorado por el center component, asi que hice un poco de hardcode poniendo 4 de padding top al Center Component para contrarrestar el 0.5 rem. (otra solucion hubiera sido sacar el margin global del toast).
PD 1: No crees que la feature de sugerencias tendria que ser separada en su archivo, teniendo tanta logica y para que sea mas facil una transicion a TypeScript?
PD 2: Crees que sea necesario aplicar Prettier al proyecto para que quede todo mas ordenado y legible, o preferis el formateo actual del proyecto?