search

FdelMazo / FIUBA-Map

Mapa de materias y sus correlativas de la Facultad de Ingeniería
https://fede.dm/FIUBA-Map/
MIT License
129 stars 37 forks source link

Fix sugerencias toast toggling #199

Closed lopezac closed 7 months ago

lopezac commented 7 months ago

Hola buenas tardes.

Lo principal de este pull request, es que el toast de sugerencias, una vez abierto no se podia cerrar, asi que agregue un state para guardar el estado de si esta abierto o cerrado y abrirlo y cerrarlo segun este state.

Tambien puse las configuraciones del toast de sugerencias al comienzo para que sea mas legible a simple vista, porque estaba mas de 150 lineas por abajo.

FdelMazo commented 7 months ago

Hola @lopezac ! Gracias por el PR, lo pulleo y reviso mas tarde. No entendí algo, el toast no se podía cerrar? Debería tener una cruz arriba a la derecha!

lopezac commented 7 months ago

Hola, me refiero a que creo que estaria bueno que el toast se pueda cerrar tambien con boton de sugerencias (con el mismo boton que se usa para abrirlo)

FdelMazo commented 7 months ago

Ah, magnifico @lopezac, gracias por la idea y el cambio. Antes de mergear, no hay alguna forma de hacerlo sin agregar una variable de estado extra? Me parece que si usas if (bugToast.current) te debería detectar que el toast esta abierto, puede ser? O tal vevz con toast.isActive()