[WIP] Attempt to make extension work with notebook 5.6

Makefile

@@ -37,7 +37,6 @@ env: ## Make a dev environment
 		pip install -e . && \
 		jupyter dashboards quick-setup --sys-prefix && \
 		jupyter nbextension enable --py widgetsnbextension --sys-prefix
-		#jupyter declarativewidgets quick-setup --sys-prefix
 
 js: ## Make JavaScript assets
 	npm install

bower.json

@@ -15,7 +15,7 @@
     "tests"
   ],
   "dependencies": {
-    "gridstack": "0.2.4",
+    "gridstack": "0.4.0",
     "requirejs": "~2.1.20",
     "requirejs-text": "~2.0.14"
   }

jupyter_dashboards/nbextension/notebook/dashboard-common/gridstack-custom.js

@@ -15,29 +15,29 @@ define(['jquery', 'Gridstack'], function($, Gridstack) {
     GridstackCustom.prototype = Object.create(Gridstack.prototype);
     GridstackCustom.prototype.constructor = GridstackCustom;
 
-    GridstackCustom.prototype.__destroy = Gridstack.prototype.destroy;
-    GridstackCustom.prototype.destroy = function(detach_node) {
-        detach_node = typeof detach_node === 'undefined' ? true : detach_node;
+    // GridstackCustom.prototype.__destroy = Gridstack.prototype.destroy;
+    // GridstackCustom.prototype.destroy = function(detach_node) {
+    //     detach_node = typeof detach_node === 'undefined' ? true : detach_node;
 
-        if (!detach_node) {
-            // Gridstack.destroy() removes the container from the DOM. Hack around it to make it
-            // a no-op if `detach_node` is false.
-            var $container = this.container;
-            var oldRemove = $.fn.remove;
-            $.fn.remove = function() {
-                if (!this.is($container)) {
-                    return oldRemove.apply(this, arguments);
-                }
-                return this;
-            };
-        }
+    //     if (!detach_node) {
+    //         // Gridstack.destroy() removes the container from the DOM. Hack around it to make it
+    //         // a no-op if `detach_node` is false.
+    //         var $container = this.container;
+    //         var oldRemove = $.fn.remove;
+    //         $.fn.remove = function() {
+    //             if (!this.is($container)) {
+    //                 return oldRemove.apply(this, arguments);
+    //             }
+    //             return this;
+    //         };
+    //     }
 
-        this.__destroy();
+    //     this.__destroy();
 
-        if (!detach_node) {
-            $.fn.remove = oldRemove;
-        }
-    };
+    //     if (!detach_node) {
+    //         $.fn.remove = oldRemove;
+    //     }
+    // };
 
     /**
      * Generates cell styles which depend on margin (which is a layout option).

jupyter_dashboards/nbextension/notebook/dashboard-view/layout/grid/layout.js

@@ -446,8 +446,8 @@ define([
     GridLayout.prototype._hideCell = function($cell) {
         var self = this;
         this.$container.one('change', function() {
-            $cell.resizable('destroy');
-            $cell.draggable('destroy');
+            $cell.resizable(false);
+            $cell.draggable(false);
             Metadata.hideCell($cell);
             // Temporarily set 'top' *before* removing 'grid-stack-item' class. This makes it so
             // cell animates when moving from dashboard to hidden cells area.
@@ -665,7 +665,7 @@ define([
         });
 
         $('.grid-stack-item')
-                .resizable('destroy').draggable('destroy')
+                .resizable(false).draggable(false)
                 .removeClass('ui-resizable-autohide'); // jquery bug, cannot remove class using API
 
         $('.dashboard-item-background').remove();