Fix production config

pd4d10 · Aug 2, 2018 · 6dd8b19 · 6dd8b19
1 parent 8eee058
commit 6dd8b19
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 17 deletions.
diff --git a/src/content-script/renderer.ts b/src/content-script/renderer.ts
@@ -1,4 +1,4 @@
-import { debounce } from 'lodash'
+import * as debounce from 'lodash/debounce'
 import { LineAndCharacter } from 'typescript'
 import { renderToDOM } from './containers'
 import {
@@ -150,7 +150,7 @@ export default class Renderer {
             definition: {
               isVisible: true,
               height: this.line.height,
-              width: this.line.width - 10, // TODO: Magic number
+              width: this.line.width - 20, // TODO: Magic number
               top: response.info.line * this.line.height,
             },
           })

diff --git a/webpack.config.production.js b/webpack.config.production.js
@@ -7,20 +7,22 @@ module.exports = {
   ...config,
   mode: 'production',
   devtool: false,
-  plugins: [
-    ...config.plugins,
-    new UglifyJsPlugin({
-      uglifyOptions: {
-        compress: {
-          drop_console: true,
+  optimization: {
+    noEmitOnErrors: false,
+    minimizer: [
+      new UglifyJsPlugin({
+        uglifyOptions: {
+          compress: {
+            drop_console: true,
+          },
+          output: {
+            // Fix Safari error:
+            // SyntaxError: Invalid regular expression: missing terminating ] for character class
+            ascii_only: process.env.TARGET === 'safari',
+          },
         },
-        output: {
-          // Fix Safari error:
-          // SyntaxError: Invalid regular expression: missing terminating ] for character class
-          ascii_only: process.env.TARGET === 'safari',
-        },
-      },
-    }),
-    new BundleAnalyzerPlugin(),
-  ],
+      }),
+    ],
+  },
+  plugins: [...config.plugins, new BundleAnalyzerPlugin()],
 }