fix: we should remove all event listeners when calling destroy (#556)

- this is to avoid having events leaking in memory from possible detached elements
6pac · Nov 26, 2020 · 708b070 · 708b070
1 parent 69a6b86
commit 708b070
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/slick.grid.js b/slick.grid.js
@@ -2090,7  2090,22 @@ if (typeof Slick === "undefined") {
       $container.off(".slickgrid");
       removeCssRules();
 
-      $canvas.off("draginit dragstart dragend drag");
       $canvas.off();
       $viewport.off();
       $headerScroller.off();
       $headerRowScroller.off();
       if ($footerRow) {
         $footerRow.off();
       }
       if ($footerRowScroller) {
         $footerRowScroller.off();
       }
       if ($preHeaderPanelScroller) {
         $preHeaderPanelScroller.off();
       }
       $focusSink.off();
       $(".slick-resizable-handle").off();
       $(".slick-header-column").off();
       $container.empty().removeClass(uid);
       if (shouldDestroyAllElements) {
         destroyAllElements();