search

FdelMazo / FIUBA-Plan

Organizador de horarios de la Facultad de Ingenieria
https://fede.dm/FIUBA-Plan/
MIT License
45 stars 6 forks source link

Implement search materias by weekday feature #106

Closed lopezac closed 5 months ago

lopezac commented 5 months ago

closes #67

Hola @FdelMazo , vi el issue #67 y me parecio muy copada la idea.

La que se me ocurrio seria que en el drawer hubiera un boton que este seleccionado por defecto en la opcion "Todos los dias" (siendo este el funcionamiento clasico), y al tocar el boton aparecieran una lista con los dias de la semana (lunes a sabado) y al seleccionar un dia, luego cuando uno busque materias aparecieran solamente las materias que tengan un curso con clases el dia seleccionado.

Quiza podria haber otro boton con el mismo funcionamiento pero de cursos, que uno al seleccionar una materia para ver sus cursos, aparecieran solamente los cursos que tengan clases el dia seleccionado.

FdelMazo commented 5 months ago

La que se me ocurrio seria que en el drawer hubiera un boton que este seleccionado por defecto en la opcion "Todos los dias" (siendo este el funcionamiento clasico), y al tocar el boton aparecieran una lista con los dias de la semana (lunes a sabado) y al seleccionar un dia, luego cuando uno busque materias aparecieran solamente las materias que tengan un curso con clases el dia seleccionado.

Esta buena la idea. Yo más que seleccionar de a un día, pensaba algo parecido a lo que hace el SIU, que tiene botones para todos los dias y vos seleccionas los que queres ver. O bueno, se puede hacer con un select y que tenga multiples elecciones. No se bien cual puede ser la mejor UX para esto.

image

Quiza podria haber otro boton con el mismo funcionamiento pero de cursos, que uno al seleccionar una materia para ver sus cursos, aparecieran solamente los cursos que tengan clases el dia seleccionado.

Para mí esto va a sobrar. Ya en cada curso tenemos el tag que te dice los dias de cursada, y me parece que entre el selector de dias de materias y eso ya va a ser bastante facil elegir que cursar

image

lopezac commented 5 months ago

Intento implementar la feature con la UX de un select donde te permite elegir que dias queres, teniendo por defecto seleccionado todos los dias.

Si sobra lo de cursos.

lopezac commented 5 months ago

@FdelMazo como te parece que esta implementado por ahora la ux?.

Utilice el clasico position absolute y relative para posicionar la List donde estan los cursos, para que tome todo el ancho del drawer, queda bien pero se posiciona arriba de las materias elegidas, tendre que ver como solucionar eso, (usando grids quiza?).

Otra cosa importante, crees que haga falta guardar los dias que el usuario selecciona para filtrar las materias?, o que este bien asi que cada vez que se refresca la pagina se borre esa info.

lopezac commented 5 months ago

Hola @FdelMazo , tus reviews son oro, como te parece que quedo ahora, ves algo para cambiar?

No encontre una forma sin usar un state de filteredMaterias, y use un Portal de chakra porque sino se quedaba la lista de los dias abajo de las materias seleccionadas.

lopezac commented 5 months ago

Si @FdelMazo muy bueno tu razonamiento de que la feature de filtrar materias no necesita ser un state al no almacenar nada , sino siendo logica.

Te parecen bien los margins de las cosas en el drawer?.

Te parece ahora que los usuarios puedan discernir facilmente si filtraron o no con estos iconos?.

lopezac commented 5 months ago

Asi te parece que quedo bien de limpio el historial del git @FdelMazo ?

FdelMazo commented 5 months ago

Perfectoo