-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convertir materias optativas en nodos #206
base: master
Are you sure you want to change the base?
Conversation
Lo veo el finde! |
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 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 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 |
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) |
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 |
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.