Skip to content

Conversation

@Archer-Key
Copy link
Collaborator

@Archer-Key Archer-Key commented Jan 3, 2026

Change Summary

  • Added timetable and timetable related components
  • Added timetable functionality (displays tasks assigned times in grid like format)
  • Added handling of timetable clashes

Change Form

  • The pull request title has an issue number
  • The change works by "Smoke testing" or quick testing
  • The change has tests
  • The change has documentation
    • Comments are included on lines that are not immediately obvious
    • Functions and variables are given long descriptive names where required

Remaining Bugs (Minor)

  • Sticky positioning does not work at all zoom levels
  • Time column does not have a dividing line
  • The tool tip does not reduce its width to being within the timetable area if it overflows

Other

  • Removed the tasks on the side bar to be easier to finish this project before the deadline

Related issue

@Precipicee
Copy link
Collaborator

I'm happy with the code in general, and from your demonstrations I've seen it working well (aside from the remaining bugs you've already mentioned). I just want to check my understanding of your structure to ensure I get what your section is doing:

schedule.tsx: The full webpage - it contains everything, and handles all actions on the page.
timetable.tsx: The container for the whole timetable.
timetable_column.tsx: Contains the Mon-Sun, and contains rows for each of those (via timetable_slot.tsx).
timetable_slot.tsx: Contains the rows
timetable_task.tsx: Contains all the task information (timetable. Handles placement on the timetable container. Contains the time_tag and topic_tag also.
time_tag.tsx: A component to show the time for a given task.
topic_tag.tsx: Same as time_tag but for topics.
time_indicator.tsx: Shows what the current time is and displays that inside the timetable.

@Archer-Key
Copy link
Collaborator Author

Basically right, I'll go through and add some documentation to the components and add some docs at the top of each file to explain what's contained in.

@Precipicee
Copy link
Collaborator

Okay, that's good. I'm happy to merge this now :)

@Precipicee Precipicee merged commit 240cf38 into use_uv Jan 3, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Timetable Page

3 participants