Change to transpiled ESM

stefcameron · stefcameron · commit 2cec08e4b0f8 · 2020-10-07T13:19:50.000-05:00
diff --git a/.changeset/orange-files-doubt.md b/.changeset/orange-files-doubt.md
@@ -2,4 +2,4 @@
 'focus-trap': patch
 ---
 
-Fix #172 (again): Add `browser` target in package.json for use by bundlers. Both Webpack and Rollup can use this target above `module` and `main`, when specified, and our UMD bundle is transpiled down to basic ES5 for maximum browser compatibility. Our ESM bundle targets ESM browsers (which means is does not support IE 11). Webpack does this [by default](https://webpack.js.org/configuration/resolve/#resolvemainfields). Rollup, however, needs to be [configured](https://github.com/rollup/plugins/tree/master/packages/node-resolve#mainfields).
+Fix #172 (again): Transpile ESM bundle down to the same browser target used for the CJS and UMD bundles. ESM is just the module system, not the browser target.
diff --git a/README.md b/README.md
@@ -48,10 +48,6 @@ Why?
 Because this module uses [`EventTarget.addEventListener()`](document.createElement('button')).
 And its only dependency, tabbable, uses [a couple of IE9+ functions](https://github.com/davidtheclark/tabbable#browser-support).
 
-> __NOTE:__ The package contains 3 entries/bundles for bundlers like Webpack and Rollup: CJS (`main`), UMD (`browser`), and ESM (`module`). Our CJS and UMD bundles are transpiled down to basic ES5 and will support IE 9+. Our ESM bundle, however, targets ESM browsers, which means it does not support IE at all because IE never supported ESM.
->
-> Both Webpack and Rollup can use the `browser` target above `module` and `main`, when specified. Webpack does this [by default](https://webpack.js.org/configuration/resolve/#resolvemainfields). Rollup, however, needs to be [configured](https://github.com/rollup/plugins/tree/master/packages/node-resolve#mainfields).
-
 ## Usage
 
 ### createFocusTrap(element[, createOptions])
diff --git a/babel.config.js b/babel.config.js
@@ -14,13 +14,13 @@ module.exports = {
       // ESM browsers for ESM builds
       presets: [
         [
+          // NOTE: With no targets specified, @babel/preset-env will transform all
+          //  ECMAScript 2015+ code by default, which is the original preset prior
+          //  to upgrading to Babel 7
           // @see https://babeljs.io/docs/en/babel-preset-env#targets
           '@babel/preset-env',
           {
             modules: false, // preserve ES modules
-            targets: {
-              esmodules: true,
-            },
           },
         ],
       ],
diff --git a/package.json b/package.json
@@ -4,7 +4,6 @@
   "description": "Trap focus within a DOM node.",
   "main": "dist/focus-trap.js",
   "module": "dist/focus-trap.esm.js",
-  "browser": "dist/focus-trap.umd.js",
   "types": "index.d.ts",
   "sideEffects": false,
   "files": [
