Merge pull request #369 from matestack/sf/es5-javascript

ES5 transpiling

Author: jonasjabari

app/javascript/packs/matestack-ui-core.js

@@ -1,2 +1,8 @@
-require('../matestack-ui-core/index')
-module.exports = MatestackUiCore
+// // Add polyfills for the asset pipeline
+// // https://github.com/matestack/matestack-ui-core/issues/238#issuecomment-583905838
+//
+// import 'core-js/stable'
+// import 'regenerator-runtime/runtime'
+
+import '../matestack-ui-core/index'
+export default MatestackUiCore

builder/config/webpack/production.js

@@ -12,5 +12,15 @@ environment.config.set("output.filename", chunkData => {
 const miniCssExtractPlugin = environment.plugins.get('MiniCssExtract')
 miniCssExtractPlugin.options.filename = "[name].min.css"
 
+// In order to have babel's es5 (ie11) transpiling be applied to all of the
+// js code, it needs to be included in the loader paths.
+// https://github.com/rails/webpacker/blob/master/docs/webpack.md
+// https://github.com/matestack/matestack-ui-core/issues/238
+//
+const babelLoader = environment.loaders.get('babel')
+delete babelLoader.exclude
+const path = require('path')
+babelLoader.include.push(path.resolve(__dirname, '../../../node_modules'))
+babelLoader.include.push(path.resolve(__dirname, '../../../app/concepts/matestack/ui/core'))
 
 module.exports = environment.toWebpackConfig()

builder/config/webpacker.yml

@@ -11,7 +11,7 @@ default: &default
 
   # Additional paths webpack should lookup modules
   # ['app/assets', 'engine/foo/app/assets']
-  resolved_paths: []
+  resolved_paths: ['./node_modules']
 
   # Reload manifest.json on all requests so we reload latest compiled packs
   cache_manifest: false