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

Update UI #197

Closed Igris-1 closed 8 months ago

Igris-1 commented 8 months ago

La aplicación esencialmente sigue igual, los cambios fueron agregar algunos margins y rounded corners a las siguientes secciones:

Web capture_35-12-2023_181023_localhost

Cosas mínimas pero que cambian la visual bastante

Respecto al siguiente código comentado, siento que con la configuración correcta no sería tan laguero, con un spacing de 24/25 y un extent factor de 1.5/2 lo dejaría pasar, el efecto snowfall navideño creo que es peor jajaja

   network.on("beforeDrawing", function (ctx) {
   var width = ctx.canvas.clientWidth;
   var height = ctx.canvas.clientHeight;
   var spacing = 22;
   var gridExtentFactor = 1;
   ctx.fillStyle = "lightgray"

   for (var x = -width * gridExtentFactor; x <= width * gridExtentFactor; x += spacing) {
     for (var y = -height * gridExtentFactor; y <= height * gridExtentFactor; y += spacing) {
       ctx.beginPath();
       ctx.arc(x, y, 1, 0, 2 * Math.PI, false);
       ctx.fill();
     }
   }
 });
FdelMazo commented 8 months ago

Gracias @Igris-1 por el PR! No soy extremadamente fan de los bordes redondeados, pero mañana reviso bien todo y veo que me gustaría incluir y que no. Tambien me fijo como afecta en mobile, y si hay alguna dif entre chrome/firefox. Gracias!!

Y si, el efecto de nieve es muy lento, creo que en todos los browsers. Si se te ocurre alguna alternativa, la agradezco!

Igris-1 commented 8 months ago

Estuve probando otros tipos de snowfall y no hay caso, son todos lentos por igual o peores, a lo mucho se podría configurar el actual sacando la rotación, reduciendo la velocidad y cantidad de los copitos, también bajandole un cambio al viento, son opciones que no afectan mucho pero están

FdelMazo commented 8 months ago

@Igris-1, no voy a incluir del todo este PR. Agradezco mucho que lo hayas hecho, pero los cambios estéticos no son mucho del estilo que le quiero dar a la herramienta.

Lo que si me encantó es el border radius en las barritas de progreso. Te parece si rearmas el PR para que sea solo el cambio de agregarle a <Progress/> un margin right de 1, y un border radius de 3 (en vez de tu propuesta de 4)?

Igris-1 commented 8 months ago

Listooo ah y me encanto el nuevo botoncito para activar/desactivar el snowfall!!