Closed Diegothx closed 11 months ago
kovaxis
commented
11 months ago Epico.
Comentario: No es redundante el "Ver equivalencias" con el ícono de lápiz para editar la equivalencia? Creo que es menos confuso tener 1 sola forma de llegar a ese menú.
También, dejo por escrito que la conclusión fue no implementar #360 por ahora, porque resulta un poco confuso de usar. En su lugar, implementar #354. También, #318 queda cerrado con este PR.
Diegothx
commented
11 months ago Epico.
Comentario: No es redundante el "Ver equivalencias" con el ícono de lápiz para editar la equivalencia? Creo que es menos confuso tener 1 sola forma de llegar a ese menú.
También, dejo por escrito que la conclusión fue no implementar #360 por ahora, porque resulta un poco confuso de usar. En su lugar, implementar #354. También, #318 queda cerrado con este PR.
Eliminado el ver equivalencias por ahora
kovaxis
commented
11 months ago El código se ve bueno, encontré algunos problemas probando si:
En Filosofía: Para Qué aparece dos veces Área Libre:
Esto me tinca mala mía, voy a revisarlo mañana.
Se me acabó la RAM y tengo demasiado sueño para esperar los 3 minutos que toma abrir todo de nuevo, pero cuando se hace click en una equivalencia y solo está la opción "Eliminar", el borde superior es más ancho de lo que debiera ser.
Creo que no se debiera mostrar el "Ver bloques disponibles" cuando solo hay 1 opción. Quizás lo cambiaría a "Asignado a X" cuando hay solo 1 bloque posible y está asignado, y "Asignar a" cuando hay más de 1 bloque posible o no está asignado.
Por último, quizás agregaría un poco de border-radius, todo el resto de la interfaz tiene border-radius.
kovaxis
commented
11 months ago Cambié lo que dije, voy a mergear, pero cualquier cosa lo revertimos si está mal.
Quedó así:
¿Qué se implementa?
Lado bueno, estoy casi seguro que ya no existe ningún softBlock en el planner, lado malo, puede resultar bastante confuso el intercambio de bloques. Un ejemplo: se puede intercambiar un curso a un bloque que es válido, pero no suma al planner (como OPIs en caso de que ya se tenga los créditos del título ocupados), lo cual permite cambiar a un curso que no sirve, quedando con una equivalencia en gris. De todos modos, ante cualquier error en el planner el autofix recomienda al tiro una solución y resulta fácil de arreglar, pero igual se puede ir puliendo. Closes #352
Único detalle, son 2 clics. De todos modos, creo que estaría bien así, puesto que obedece la lógica de "un curso extra será un curso extra a menos que el usuario fuerce lo contrario". De todos modos, se podría implementar que, si se detecta que un curso extra puede reemplazar una equivalencia no definida, esto se realice automáticamente. Closes #319
No implemente #318 como tal, pero en "ver Bloques posibles" se marca el bloque actual. Si les convence lo dejamos así y si no puedo agregar el tooltip de todos modos. Sí es así entonces Closes #318
Aproveche que linkee los programas al contextMenu e hice lo mismo dentro del selector de curso para equivalencias. No es exactamente #128 pero resulta suficiente según yo. Igual se puede evaluar él agregar una vista propia del planner si lo consideran una mejor idea.
Para tener el mismo formato al equipo y a ls contribuyentes actualicé la parte del equipo del README a
si no les convence bienvenids a cambiarlo de vuelta.
¿Qué falta? (Futuras PR):
Prerrequisitos
semester-credspensando en que se cerrarían los PR antes de terminar con esta, además así mantenía la lógica sincronizada a como funciona el planner actualmente