Timeline.destroy works

HISTORY.md

@@ -6,6 +6,7 @@ http://visjs.org
 
 ### Timeline
 
+- Implemented function `destroy` to neatly cleanup a Timeline.
 - Implemented support for dragging the timeline contents vertically.
 - Implemented options `zoomable` and `moveable`.
 - Changed default value of option `showCurrentTime` to true.

docs/timeline.html

@@ -673,6 +673,13 @@ timeline.clear({options: true}); // clear options only
     </td>
   </tr>
 
+  <tr>
+    <td>destroy()</td>
+    <td>none</td>
+    <td>Destroy the Timeline. The timeline is removed from memory. all DOM elements and event listeners are cleaned up.
+    </td>
+  </tr>
+
   <tr>
     <td>fit()</td>
     <td>none</td>

src/timeline/Timeline.js

@@ -227,7 +227,7 @@ Timeline.prototype.destroy = function () {
   // cleanup hammer touch events
   for (var event in this.listeners) {
     if (this.listeners.hasOwnProperty(event)) {
-      this.hammer.off(event, this.listeners[event]);
+      delete this.listeners[event];
     }
   }
   this.listeners = null;
@@ -238,17 +238,6 @@ Timeline.prototype.destroy = function () {
     component.destroy();
   });
 
-  // remove all components
-  this.components = [];
-  this.range = null;
-  this.timeAxis = null;
-  this.itemSet = null;
-  this.currentTime = null;
-  this.customTime = null;
-
-  //this.body.util.snap = null;
-  //this.body.util.toTime = null;
-  //this.body.util.toScreen = null;
   this.body = null;
 };
 

src/timeline/component/ItemSet.js

@@ -296,7 +296,6 @@ ItemSet.prototype.destroy = function() {
   this.setItems(null);
   this.setGroups(null);
 
-  this.hammer.enable(false);
   this.hammer = null;
 
   this.body = null;