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

Convertir materias optativas en nodos #206

Open lopezac opened 5 months ago

lopezac commented 5 months ago

closes #152

Hola @FdelMazo como andas, me parecio bastante buena la idea del issue.

Lo que se ocurre seria crear un rombo (como el de CBC) llamado "Optativas", junto al rombo del CBC.

Cuando el usuario añade materias optativas crear nodos con el nombre que le puso (Motivo), y que esten linkeados a este rombo Optativas, que sean de color rojo (o el que te guste). Tendria que funcionar muy similarmente al del CBC.

Este rombo solamente tendria que aparecer si el usuario tiene optativas.

En la barrita del footer de electivas, dejar de color violeta la cantidad de creditos de las electivas reales, y poner de color rojo la cantidad de creditos de las optativas.

Que opinas?, deje algunas sugerencias en forma de comentarios en el commit, si no te copan los borro.

FdelMazo commented 5 months ago

Lo veo el finde!

FdelMazo commented 5 months ago

Buenas! Si, me parece bien empezar la idea. No se si hace falta que estén abajo de un rombo como el del CBC, porque algo importante que le veo a las materias optativas es que también tengan su propio cuatrimestre, como el resto de los nodos (y hay que ver como persistir eso). El rombo me parece que es más como para decir "de aca para atras tenes escondidas mas materias", y acá me parece que queda mejor solamente tener nodos nuevos dando vueltas en el grafo. Tal vez solamente usando un [Optativa] abajo del nombre de la materia ya basta para diferenciar el nodo en el grafo.

Creo que es una funcionalidad que va a traer distintos matices a medida que se codee, asi que mientras mas se avance mas facil va a ser repensarla.

Con lo que hay que tener cuidado es que muchas veces en los getters se llama a todos los nodos y se contabilizan sus creditos, pero ya en este momento tenemos la contabilizacion de los creditos de las optativas por separado, entonces hay que tener cuidado con no doble-contar los creditos en la barrita de progreso.

Lo de la barrita del footer me es indistinto, si queda lindo genial, y si no con solamente cambiar el titulo de las materias electivas a materias electivas y optativas ya esta.

Y en cuanto a los comentarios del commit que ya hay en este PR, creo que no son relevantes a esta funcionalidad (y tampoco se si son necesarios de aplicar), tal vez pueden quedar para algún refactor mas adelante.


De todas formas, creo que es una funcionalidad grande y con varias cosas para programar. Siempre se pueden setupear las bases de la idea, y dejarlo para que otra persona haga el PR

lopezac commented 5 months ago

Si @FdelMazo , creo que lo mejor y mas sencillo seria:

Que opinas Fede?, hay algo que cambiar o agregar?, (y luego sacar los comentarios del commit)

FdelMazo commented 5 months ago

Si, genial eso. Yo no cambiaria lo de "electivas" a "electivas y optativas", y en el tooltip hay que acordarse de agregar el "y optativas" solamente si el usuario tiene optativas. Lo veo todo bien