search

gordon-cs / gordon-360-api

The 360° Gordon Experience
11 stars 6 forks source link

Term selector order by most recent first and show current #990

Closed russtuck closed 3 months ago

russtuck commented 1 year ago

The new term selector shows terms from oldest to newest, but the current term is the most frequently used. Ordering from newest to oldest will put the current term at or near the top, making the common case quicker.

EjPlatzer commented 1 year ago

To add on to this issue, it seems intuitive to me that the Schedule panel should show the schedule of most recent term by default

russtuck commented 1 year ago

Good point, and there's currently a bug (in train) where the schedule is empty until a term is selected. The schedule is labeled with the current term, but empty. This is more obvious in train, where the current term is spring 2021-22, than in production (where it's currently summer and many people have an empty schedule anyway).

And there's a related issue: clicking the pulldown is now slow (because it's doing a lot of work), and there's no "loader" image to indicate it's working. Showing the current schedule by default should fix this: it will show a loader while it fetches the schedule, and then the info for the pulldown will be available.

russtuck commented 1 year ago

See comment on #981 about fixing order in the API.

The loading issues are in the UI, of course.

Luke-W-Hart commented 3 months ago

Dr. Tuck: "Evan fixed that (loading issues on the pulldown) quickly after Summer Practicum '23 was over".