Skip to content

adds tickCap, cleans up timer module, adds docs - #97

Open
mgh wants to merge 2 commits into
masterfrom
feature-timer
Open

mgh wants to merge 2 commits into
masterfrom
feature-timer

Conversation

@mgh

@mgh mgh commented Jan 28, 2016

Copy link
Copy Markdown
Contributor

tickCap allows applications to specify a cap for tick values
separately from the maxTick property.

cleanup:

  • timer.maxTick can now be set to control the value of maxTick
  • timer.debug was broken and unused, now it is removed
  • global variable ok was removed
  • jsdoc strings added

Martin Hunt added 2 commits January 28, 2016 17:39
`tickCap` allows applications to specify a cap for tick values
separately from the maxTick property.

cleanup:
 - `timer.maxTick` can now be set to control the value of `maxTick`
 - `timer.debug` was broken and unused, now it is removed
 - global variable `ok` was removed
 - jsdoc strings added
Comment thread src/timer.js
* - `100 ⟶ onTick(100)`
* - `120 ⟶ onTick(100)`
* - `9999 ⟶ onTick(100)`
* - `10000 ⟶ onTick(10000)`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this supposed to be onTick(100) as well?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants