search

FdelMazo / FIUBA-Map

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

Limitar el des-zoom. #182

Closed LucasDondo closed 7 months ago

LucasDondo commented 10 months ago

¡Buenas Fede crack! Me imagino que a muchos nos gusta ir poco a poco subiendo la sensibilidad del mouse hasta que cuando apenas lo acaricies ya dé 3 vueltas olímpicas a la pantalla. El problema es que eso no es tan compatible con FIUBA Map todavía... (O más bien la sensibilidad de "la ruedita" del mouse) Porque al ampliar (bajar mucho el zoom) al final hasta perdes de vista el mapa. Estaría bueno que el des-zoom solo te permita hacerlo hasta cierto punto (ponele, hasta el punto el cual ya se ven todas las materias).

https://github.com/FdelMazo/FIUBA-Map/assets/59975541/2b26c0ba-d243-4308-80f2-7c73d7b3df62

FdelMazo commented 10 months ago

Uh, la verdad que es una buena idea. Creo que visjs no tiene forma nativa de hacerlo, pero se puede hacer con un listener al evento de zoom. Esto se puede setupear en createNetwork.

Estaría bueno limitar tanto el zoom como el deszoom, y encontrarle algun valor sensible a cada uno.

Por otro lado, en tu video veo que el texto del CBC esta en negro, incluso en el dark mode. Eso no debería pasar! En que browser estas? Me sorprenden todos los colores de tu video, lo veo muy azulado. Tenes el contraste alto?

Así lo veo yo. image

LucasDondo commented 10 months ago

Estaría bueno limitar tanto el zoom como el deszoom, y encontrarle algun valor sensible a cada uno.

Banco!

Por otro lado, en tu video veo que el texto del CBC esta en negro, incluso en el dark mode. Eso no debería pasar! En que browser estas? Me sorprenden todos los colores de tu video, lo veo muy azulado. Tenes el contraste alto?

Ah, no, es que uso Dark Reader. Jaja ahí lo desactivé para esa página. Buena observación!

FdelMazo commented 7 months ago

@LucasDondo, limité el zoom en 0f4a719115b00ff404e447024583876ab53be3f0

Queres testearlo en distintas carreras y si te parece bien implementado, cerrar el issue? Gracias!!

LucasDondo commented 6 months ago

Uh, nunca me llegó ninguna notificación ni nada... Vi que ya lo implementaste, ¡quedó joya! ¡Gracias Fede!