minify js files (#7)

* minify js files

* install jsmin

* fix bug

* bugfix

Author: njzjz

README.md

@@ -5,7 +5,7 @@ This package should be used in all sphinx projects under the [@deepmodeling](htt
 ## Features
 
 - Adding the DeepModeling banner
-- Minify HTML files
+- Minify HTML and Javascript files
 
 ## How to use it
 

deepmodeling_sphinx/inject.py

@@ -3,6 +3,7 @@
 import os
 
 import htmlmin
+from jsmin import jsmin
 from sphinx.application import Sphinx
 from sphinx.util.fileutil import copy_asset_file
 
@@ -72,9 +73,20 @@ def render(self, template, render_context):
                                                         app.builder.templates)
 
 
+def minify_js_files(app, exception):
+    for js in app.builder.script_files:
+        fn = os.path.join(app.builder.outdir, js)
+        if os.path.isfile(fn):
+            with open(fn, 'r+') as f:
+                minified_js = jsmin(f.read())
+                f.seek(0)
+                f.write(minified_js)
+                f.truncate()
+
+
 def setup(app: Sphinx) -> Dict[str, Any]:
     app.connect('builder-inited', copy_custom_files)
     app.connect('html-page-context', insert_sidebar)
-    app.connect('html-page-context', minify_html_files)
+    app.connect('build-finished', minify_js_files)
 
     return {'parallel_read_safe': True}

setup.py

@@ -8,6 +8,7 @@
     install_requires=[
         'sphinx',
         'htmlmin',
+        'jsmin',
     ],
     package_data={
         'deepmodeling_sphinx': ['banner.html',