compsci-adl / mytimetable

A drag-and-drop timetable planner for University of Adelaide students.
https://mytimetable.csclub.org.au
MIT License
9 stars 2 forks source link

[Feature Request] Show course conflicts #5

Open jsun969 opened 1 month ago

jsun969 commented 1 month ago

Overview

Show conflicts between courses in both chips and calendar.

Details

For chip and course modal: add a :warning: emoji in chip and show conflict details in course modal

For calendar: conflict course cards should be displayed side by side like CleanShot 2024-09-23 at 12 35 21@2x

There is a timeOverlap util function (utils/time-overlap-ts) which may be helpful.

Remove course sorting in getWeekCourses (helpers/calendar.ts) and zIndex in CalendarCourses (components/Calendar.tsx) since it's for displaying conflicts by opacity.

Why would this feature be useful?

Tell user which courses are conflict when scheduling timetable

Additional Notes

No response

jsun969 commented 1 month ago

Fixed the conflicts in calendar when times are same. But overlapping time is still an issue

CleanShot 2024-09-25 at 15 45 28@2x

cyruskelly commented 4 weeks ago

If there is a conflict between a course and an immovable course the immovable course will not display correctly:

image image
jsun969 commented 4 weeks ago

@cyruskelly

I think it's correctly displayed for now. Since courses with longer duration will be at bottom to make sure user can drag courses that have shorter duration, they will be covered.

But it will be better if we can solve this issue and display conflicts like apple calendar.