-
Notifications
You must be signed in to change notification settings - Fork 1
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
Rename current timer class and add actual custom timer setup #41
Comments
renamed class in 2855f7f |
Should the custom timer also write to the |
I think the solution will be to have one set of timer controls on the page. However, there will be two input boxes; one for the 'lobby timer' and the other for the 'custom timer' (would probably just op to name them timer 1 & 2, as there is no need to associate one or the other with its task). Timer lengths can be moved to a preference based system, as this is something a user might prefer to hang around between folder changes |
see #72 |
I think some changes might need to be made to make this page look a little better and increase its clarity, but for now it's looking okay. And of course the Timer back-end is a wreck rn as it still contains code for the intended separation of the lobby and custom timer. The Lobby Timer class is the one being used currently, and the custom timer will probably be removed in future. |
A second timer so we have a "Custom" and a "Lobby". Custom timer can have easier to see setter, and the lobby can have a more permanent length
The text was updated successfully, but these errors were encountered: