feat(bundle-size): minify js, too (#67)

Author: seanforyou23

package.json

@@ -33,7 +33,7 @@
     "@types/react-dom": "^16.8.5",
     "@types/react-router-dom": "^4.3.4",
     "@types/storybook__react": "^4.0.2",
-    "@types/victory": "^31.0.14",
+    "@types/victory": "^33.0.0",
     "@types/webpack": "^4.32.1",
     "css-loader": "^3.2.0",
     "enzyme": "^3.7.0",
@@ -56,6 +56,7 @@
     "rimraf": "^3.0.0",
     "style-loader": "^1.0.0",
     "svg-url-loader": "^3.0.0",
+    "terser-webpack-plugin": "^2.1.0",
     "ts-jest": "^24.0.0",
     "ts-loader": "^6.0.2",
     "tsconfig-paths-webpack-plugin": "^3.2.0",

src/app/AppLayout/AppLayout.tsx

@@ -48,7 +48,7 @@ const AppLayout: React.FunctionComponent<IAppLayout> = ({children}) => {
     <Nav id="nav-primary-simple">
       <NavList id="nav-list-simple" variant={NavVariants.simple}>
         {routes.map((route, idx) => {
-          return (
+          return route.label && (
             <NavItem key={`${route.label}-${idx}`} id={`${route.label}-${idx}`}>
               <NavLink exact={true} to={route.path} activeClassName="pf-m-current">{route.label}</NavLink>
             </NavItem>

tsconfig.json

@@ -21,7 +21,7 @@
     "paths": {
       "@app/*": ["src/app/*"],
       "@assets/*": ["node_modules/@patternfly/react-core/dist/styles/assets/*"]
-    },
+    }
   },
   "include": [
     "**/*.ts",

webpack.prod.js

@@ -3,12 +3,14 @@ const merge = require('webpack-merge');
 const common = require('./webpack.common.js');
 const MiniCssExtractPlugin = require('mini-css-extract-plugin');
 const OptimizeCSSAssetsPlugin = require('optimize-css-assets-webpack-plugin');
+const TerserJSPlugin = require('terser-webpack-plugin');
 
 module.exports = merge(common, {
   mode: 'production',
   devtool: 'source-map',
   optimization: {
     minimizer: [
+      new TerserJSPlugin({}),
       new OptimizeCSSAssetsPlugin({})
     ],
   },