From b69509fcab081fd64fd0441552f08c623384e47e Mon Sep 17 00:00:00 2001
From: Grant Nestor <grantnestor@gmail.com>
Date: Mon, 20 Mar 2017 20:32:16 -0700
Subject: [PATCH] Move preact, preact-compat, and proptypes to bower

---
 bower.json                                    |  3 +++
 notebook/static/notebook/js/shortcuteditor.js |  2 +-
 notebook/templates/page.html                  |  4 ++--
 package.json                                  |  2 --
 setupbase.py                                  | 24 +++----------------
 5 files changed, 9 insertions(+), 26 deletions(-)

diff --git a/bower.json b/bower.json
index ee6d222f3..e58ec9279 100644
--- a/bower.json
+++ b/bower.json
@@ -15,6 +15,9 @@
     "marked": "~0.3",
     "MathJax": "components/MathJax#~2.6",
     "moment": "~2.8.4",
+    "preact": "https://unpkg.com/preact@^7.2.0/dist/preact.min.js",
+    "preact-compat": "https://unpkg.com/preact-compat@^3.14.3/dist/preact-compat.min.js",
+    "proptypes": "https://unpkg.com/proptypes@^0.14.4/index.js",
     "requirejs": "~2.1",
     "text-encoding": "~0.1",
     "underscore": "components/underscore#~1.8.3",
diff --git a/notebook/static/notebook/js/shortcuteditor.js b/notebook/static/notebook/js/shortcuteditor.js
index be0c2c434..e8e445420 100644
--- a/notebook/static/notebook/js/shortcuteditor.js
+++ b/notebook/static/notebook/js/shortcuteditor.js
@@ -5,7 +5,7 @@ define([
     "jquery",
     "notebook/js/quickhelp",
     "base/js/dialog",
-    'components/marked/lib/marked',
+    "components/marked/lib/marked"
 ], function (
     $,
     QH,
diff --git a/notebook/templates/page.html b/notebook/templates/page.html
index 94c4cd723..e40b3147b 100644
--- a/notebook/templates/page.html
+++ b/notebook/templates/page.html
@@ -16,9 +16,9 @@
     {% endblock %}
     <link rel="stylesheet" href="{{ base_url }}custom/custom.css" type="text/css" />
     <script src="{{static_url("components/es6-promise/promise.min.js")}}" type="text/javascript" charset="utf-8"></script>
-    <script src="{{static_url('components/preact/dist/preact.min.js')}}" type="text/javascript"></script>
+    <script src="{{static_url('components/preact/index.js')}}" type="text/javascript"></script>
     <script src="{{static_url('components/proptypes/index.js')}}" type="text/javascript"></script>
-    <script src="{{static_url('components/preact-compat/dist/preact-compat.min.js')}}" type="text/javascript"></script>
+    <script src="{{static_url('components/preact-compat/index.js')}}" type="text/javascript"></script>
     <script src="{{static_url('components/requirejs/require.js') }}" type="text/javascript" charset="utf-8"></script>
     <script>
       require.config({
diff --git a/package.json b/package.json
index 6b3a7aff9..c3b4ecb62 100644
--- a/package.json
+++ b/package.json
@@ -19,7 +19,5 @@
     "requirejs": "^2.1.17"
   },
   "dependencies": {
-    "preact": "^7.1.0",
-    "preact-compat": "^3.9.4"
   }
 }
diff --git a/setupbase.py b/setupbase.py
index 1c2d44b9d..afaa11645 100644
--- a/setupbase.py
+++ b/setupbase.py
@@ -147,8 +147,8 @@ def find_package_data():
         pjoin(components, "jquery-ui", "themes", "smoothness", "jquery-ui.min.css"),
         pjoin(components, "jquery-ui", "themes", "smoothness", "images", "*"),
         pjoin(components, "marked", "lib", "marked.js"),
-        pjoin(components, "preact", "dist", "preact.min.js"),
-        pjoin(components, "preact-compat", "dist", "preact-compat.min.js"),
+        pjoin(components, "preact", "index.js"),
+        pjoin(components, "preact-compat", "index.js"),
         pjoin(components, "proptypes", "index.js"),
         pjoin(components, "requirejs", "require.js"),
         pjoin(components, "underscore", "underscore-min.js"),
@@ -354,14 +354,6 @@ class Bower(Command):
         if not os.path.exists(self.bower_dir):
             return True
         
-        # check npm packages
-        for pkg in ['preact', 'preact-compat', 'proptypes']:
-            npm_pkg = os.path.join(self.node_modules, pkg)
-            bower_pkg = os.path.join(self.bower_dir, pkg)
-            if not os.path.exists(npm_pkg) or not os.path.exists(bower_pkg):
-                return True
-            if mtime(bower_pkg) < mtime(npm_pkg):
-                return True
         return mtime(self.bower_dir) < mtime(pjoin(repo_root, 'bower.json'))
 
     def should_run_npm(self):
@@ -372,16 +364,6 @@ class Bower(Command):
             return True
         return mtime(self.node_modules) < mtime(pjoin(repo_root, 'package.json'))
 
-    def npm_components(self):
-        """Stage npm frontend dependencies into components"""
-        for pkg in ['preact', 'preact-compat', 'proptypes']:
-            npm_pkg = os.path.join(self.node_modules, pkg)
-            bower_pkg = os.path.join(self.bower_dir, pkg)
-            log.info("Staging %s -> %s" % (npm_pkg, bower_pkg))
-            if os.path.exists(bower_pkg):
-                shutil.rmtree(bower_pkg)
-            shutil.copytree(npm_pkg, bower_pkg)
-
     def patch_codemirror(self):
         """Patch CodeMirror until https://github.com/codemirror/CodeMirror/issues/4454 is resolved"""
         
@@ -415,7 +397,7 @@ class Bower(Command):
             print("You can install js dependencies with `npm install`", file=sys.stderr)
             raise
         self.patch_codemirror()
-        self.npm_components()
+        # self.npm_components()
         os.utime(self.bower_dir, None)
         # update package data in case this created new files
         update_package_data(self.distribution)