-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Combining classes #16
Comments
When two "free" periods merge, which name should take precedence? When you combine Free with Lunch, which name should the merged period take on? If you call it Free, could this ever cause confusion with some students thinking Lunch is missing? If you call it Lunch, could this cause confusion about when Lunch starts for other students? The case where I think the distinction is relevant is if you have something like a club meeting at Lunch and want to know when it will start. What are your thoughts on this? |
Maybe it can be Free the whole way, but then when there is for example 5 or 10 mins left the circle is not going to reset. Let me know if you need more clarification. |
Sorry for the delay; I've been working on other issues/features before this one. In your example, what do you suggest the time displays during the 1 hour free session? Would it count down from 1 hour 30 min (to the end of Lunch) or 1 hour (to the beginning of lunch)? Either way, I'm worried this mixture between colors and circle proportion not corresponding as normal might be confusing. |
I would say to go through lunch, because that is technically free time. And would prefer that. (It could also be a subsetting if you want to have breaks like brunch/lunch be included or separate from free time.) |
It would be cool to have an option to have all of your free periods merge with other periods. Like how if you had a free 4th, it would merge with lunch and it would be called "free".
![screen shot 2017-11-01 at 1 20 45 pm](https://user-images.githubusercontent.com/13617534/32295802-90c1acba-bf07-11e7-809c-77b775695ecf.png)
The text was updated successfully, but these errors were encountered: