Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Synchronized clock #4

Merged
merged 2 commits into from Feb 7, 2012

Conversation

Projects
None yet
2 participants
Contributor

julienw commented Nov 22, 2011

with the current implementation, there is a problem when the timeout is set to 60000 (ie 1 minute) instead of the default 1000. In this case, the change (the repaint) is set to 1 minute after the startClock call, which means it will most probably not in the same time as the minute change.

This pull request tries to fix this: if the timeout is 1 minute, then we compute the timeout to be the number of seconds to the next minute.

An example file (index10.html) has been added to demonstrate this.

Contributor

julienw commented Jan 3, 2012

ping :-)

Owner

dsparling commented Feb 7, 2012

Been buried in work. I'll take a look and do a pull.

dsparling added a commit that referenced this pull request Feb 7, 2012

@dsparling dsparling merged commit 5017472 into dsparling:master Feb 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment