Fixes the pause flag logic as it was using the global (shared) state,…

… use Symbol for the paused flag , and replace function with proper getter sasha240100#21

README.md

@@ -85,15 +85,15 @@ new Between(
  [Number|Object|Array] to
 )
 
-// Chainable Methods
+// Methods
   .time([Number] duration) // Set duration
   .loop([String] mode, [?Number] repeatTimes) // Set loop mode, if "repeatTimes" is falsy, treats as "endless"
   .easing([Function] easing) // Set easing function
   .on([String] eventName, [Function] callback) // Add event listener
   .pause() // Pauses
 
-// Util Methods
-  .isPaused() // returns true if paused
+// Getters
+  .isPaused // returns true if paused
 ```
 
 

examples/visual-box.html

@@ -57,7 +57,7 @@ <h5 data-state style="padding-top: 20px;">state: started</h5>
 
       function toggle() {
         if (!between) return;
-        if (between.isPaused()) {
+        if (between.isPaused) {
           between.start();
         } else {
           between.pause();

src/between.js

@@ -9,16 +9,16 @@ const _betweens = [];
 const SYMBOL_TYPE = Symbol('type');
 const SYMBOL_START_TIME = Symbol('start_time');
 const SYMBOL_COMPLETED = Symbol('completed');
+const SYMBOL_PAUSED = Symbol('paused');
 
 const _requestAnimationFrame = ( // polyfill
   requestAnimationFrame
   || raf
 );
 
-let _prevTime = Date.now(), _time, _delta, _paused;
+let _prevTime = Date.now(), _time, _delta;
 (function _update() {
   _requestAnimationFrame(_update);
-  if (_paused) return;
 
   _time = Date.now();
   _delta = _time - _prevTime;
@@ -74,7 +74,8 @@ export default class Between extends Events {
         ),
       [SYMBOL_COMPLETED]: false,
       [SYMBOL_TYPE]: type,
-      [SYMBOL_START_TIME]: Date.now()
+      [SYMBOL_START_TIME]: Date.now(),
+      [SYMBOL_PAUSED]: false
     });
 
     switch (this[SYMBOL_TYPE]) {
@@ -130,20 +131,18 @@ export default class Between extends Events {
   }
 
   pause() {
-    _paused = true;
+    this[SYMBOL_PAUSED] = true;
     this.emit('pause', this.value, this, _delta);
     return this;
   }
 
-  isPaused() {
-    return _paused;
+  get isPaused() {
+    return this[SYMBOL_PAUSED];
   }
 
   start() {
+    this[SYMBOL_PAUSED] = false;
     this.emit('start', this.value, this, _delta);
-    _paused = false;
-    _delta = 0;
-    _prevTime = Date.now();
     return this;
   }
 
@@ -206,6 +205,9 @@ export default class Between extends Events {
     const {_updateValue} = this;
 
     return (delta, time) => {
+      if (this[SYMBOL_PAUSED])
+        return;
+
       if (this.localTime === 0)
         this.emit('start', this.value, this);
 

types/between-tests.ts

@@ -22,7 +22,7 @@ between.on('complete', (value) => { value });
 // Test pause control
 between.pause();
 between.start();
-between.isPaused();
+between.isPaused; 
 
 // Test chained
 between.time(1000)