Add timetable grid component

crux-bphc / chronofactorem-rewrite

A rewrite of our ChronoFactorem project

https://www.chrono.crux-bphc.com/

MIT License

12 stars 3 forks source link

Add timetable grid component #129

Closed skoriop closed 8 months ago

soumitradev commented 8 months ago

Beyond this, I think I should probably mention this somewhere, so I'm mentioning it in this PR. The parent page should set vertical to true automatically on any screen that is smaller than that specified by tailwind's lg size.

soumitradev commented 8 months ago

Oh and also 2 more changes:

Vertical view does not render the times, but it should. This was omitted in the old frontend because I couldn't get it to render properly by release.
The X button does not appear on devices smaller than sm. The section should have a double-click handler to substitute for the functionality lost by not rendering that button

skoriop commented 8 months ago

The X button does not appear on devices smaller than sm. The section should have a double-click handler to substitute for the functionality lost by not rendering that button.

Since the section already has a handleUnitClick click handler, I think this can be handled by the function passed in by the parent component (which is also what was done in the old frontend).