search

ucsb-cs56-w20 / ucsb-courses-search

Spring Boot App to show similar information to that found at UCSB Curriculum Search page
https://ucsb-courses-search.herokuapp.com
2 stars 5 forks source link

T 7pm a weeklyview rowspan #269

Closed dariansung closed 4 years ago

dariansung commented 4 years ago

This PR changes the visual appearance of courses on the weekly view. The courses now stretch across the time frame they occur instead of being individually listed on multiple cells.

andrewhlu commented 4 years ago

Made some changes to fix whitespace. Generally, you should follow the same conventions followed by other files. In this case, we use "Spaces: 4".

No changes are needed on your part regarding this, I'm just putting this out there as an FYI.

andrewhlu commented 4 years ago

@jdavalos1 and I have both tested the weekly view feature you have made, and it looks good to both of us! However, you do now have a merge conflict that needs to be resolved again. Could you do that and request my review one last time? I'll take one last look, accept it, and give you all your points.