-
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
Live-updating of time remaining and current time #57
Comments
For the auto-reload thing, when you're on the upload form it would be annoying however it would also let you know that times up to upload, I tried uploading on saturday but it turned midnight and didn't let me so having it automatically refresh wouldn't be any more irratating |
@RushilP That makes sense on the upload page for a non-admin, but for any other case it would be annoying. Imagine you're changing your password at the moment it refreshes (whoops there goes that). Or if you're updating a report (which you can't do yet, but I will add that in the future). So thank you for making me realize I should do that on upload for non-admins, but only in that case. Anywhere else would be a bad idea. |
It may be possible to also just update the timer using iframes (unless that is a deprecated feature), I believe some old websites did that, not sure though |
That would be an extremely ugly solution, especially when I can just set up a small piece of JS and a span with an ID. Right now however, I am much more concerned with other issues. |
Right now there are two spots where time is displayed on every page, it'd be nice to have a small bit of JavaScript updating those every second (or every minute..in fact, I think that's a better idea so it isn't so hard on mobile devices).
(Additionally, how about if time remaining goes to 0, it automatically refreshes the page? Note that there are cases where this would annoy the user, like being on the upload form.. How about instead, it changes Time Remaining to a "Next turn has started!" and a "Refresh?" link?)
The text was updated successfully, but these errors were encountered: