Merge pull request #47 from ai/size

Reduce size

Author: theKashey

.babelrc

@@ -7,7 +7,13 @@
       ],
       "plugins": [
         "@babel/plugin-proposal-class-properties",
-        "@babel/plugin-transform-runtime"
+        "@babel/plugin-transform-runtime",
+        [
+          "transform-react-remove-prop-types",
+          {
+            "mode": "wrap"
+          }
+        ]
       ]
     },
     "es2015": {
@@ -22,7 +28,13 @@
       ],
       "plugins": [
         "@babel/plugin-proposal-class-properties",
-        "@babel/plugin-transform-runtime"
+        "@babel/plugin-transform-runtime",
+        [
+          "transform-react-remove-prop-types",
+          {
+            "mode": "wrap"
+          }
+        ]
       ]
     }
   }

package.json

@@ -52,6 +52,7 @@
     "@storybook/addon-links": "^3.2.14",
     "@storybook/react": "^3.2.14",
     "babel-eslint": "^9.0.0",
+    "babel-plugin-transform-react-remove-prop-types": "^0.4.19",
     "chai": "^4.1.0",
     "chai-enzyme": "^1.0.0-beta.0",
     "enzyme": "^3.3.0",

src/AutoFocusInside.js

@@ -9,13 +9,11 @@ const AutoFocusInside = ({ disabled, children, className }) => (
   </div>
 );
 
-if (process.env.NODE_ENV !== 'production') {
-  AutoFocusInside.propTypes = {
-    children: PropTypes.node.isRequired,
-    disabled: PropTypes.bool,
-    className: PropTypes.string,
-  };
-}
+AutoFocusInside.propTypes = {
+  children: PropTypes.node.isRequired,
+  disabled: PropTypes.bool,
+  className: PropTypes.string,
+};
 
 AutoFocusInside.defaultProps = {
   disabled: false,

src/FreeFocusInside.js

@@ -9,12 +9,10 @@ const FreeFocusInside = ({ children, className }) => (
   </div>
 );
 
-if (process.env.NODE_ENV !== 'production') {
-  FreeFocusInside.propTypes = {
-    children: PropTypes.node.isRequired,
-    className: PropTypes.string,
-  };
-}
+FreeFocusInside.propTypes = {
+  children: PropTypes.node.isRequired,
+  className: PropTypes.string,
+};
 
 FreeFocusInside.defaultProps = {
   disabled: false,

src/Lock.js

@@ -75,9 +75,11 @@ class FocusLock extends Component {
     } = this.props;
     const { observed } = this.state;
 
-    if (typeof allowTextSelection !== 'undefined') {
-      // eslint-disable-next-line no-console
-      console.warn('React-Focus-Lock: allowTextSelection is deprecated and enabled by default');
+    if (process.env.NODE_ENV !== 'production') {
+      if (typeof allowTextSelection !== 'undefined') {
+        // eslint-disable-next-line no-console
+        console.warn('React-Focus-Lock: allowTextSelection is deprecated and enabled by default');
+      }
     }
 
     const lockProps = {
@@ -116,29 +118,27 @@ class FocusLock extends Component {
   }
 }
 
-if (process.env.NODE_ENV !== 'production') {
-  FocusLock.propTypes = {
-    children: PropTypes.node.isRequired,
-    disabled: PropTypes.bool,
-    returnFocus: PropTypes.bool,
-    noFocusGuards: PropTypes.bool,
+FocusLock.propTypes = {
+  children: PropTypes.node.isRequired,
+  disabled: PropTypes.bool,
+  returnFocus: PropTypes.bool,
+  noFocusGuards: PropTypes.bool,
 
-    allowTextSelection: PropTypes.bool,
-    autoFocus: PropTypes.bool,
-    persistentFocus: PropTypes.bool,
+  allowTextSelection: PropTypes.bool,
+  autoFocus: PropTypes.bool,
+  persistentFocus: PropTypes.bool,
 
-    group: PropTypes.string,
-    className: PropTypes.string,
+  group: PropTypes.string,
+  className: PropTypes.string,
 
-    whiteList: PropTypes.func,
+  whiteList: PropTypes.func,
 
-    as: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),
-    lockProps: PropTypes.object,
+  as: PropTypes.oneOfType([PropTypes.string, PropTypes.func]),
+  lockProps: PropTypes.object,
 
-    onActivation: PropTypes.func,
-    onDeactivation: PropTypes.func,
-  };
-}
+  onActivation: PropTypes.func,
+  onDeactivation: PropTypes.func,
+};
 
 FocusLock.defaultProps = {
   disabled: false,

src/MoveFocusInside.js

@@ -47,10 +47,8 @@ export default class MoveFocusInside extends Component {
   }
 }
 
-if (process.env.NODE_ENV !== 'production') {
-  MoveFocusInside.propTypes = {
-    children: PropTypes.node.isRequired,
-    disabled: PropTypes.bool,
-    className: PropTypes.string,
-  };
-}
+MoveFocusInside.propTypes = {
+  children: PropTypes.node.isRequired,
+  disabled: PropTypes.bool,
+  className: PropTypes.string,
+};

src/Trap.js

@@ -89,11 +89,9 @@ const FocusTrap = ({ children }) => (
   </div>
 );
 
-if (process.env.NODE_ENV !== 'production') {
-  FocusTrap.propTypes = {
-    children: PropTypes.node.isRequired,
-  };
-}
+FocusTrap.propTypes = {
+  children: PropTypes.node.isRequired,
+};
 
 const attachHandler = () => {
   document.addEventListener('focusin', onTrap, true);

src/util/.babelrc

@@ -0,0 +1,25 @@
+{
+  "env": {
+    "cjs": {
+      "presets": [
+        "@babel/preset-env",
+        "@babel/preset-react"
+      ],
+      "plugins": [
+      ]
+    },
+    "es2015": {
+      "presets": [
+        [
+          "@babel/preset-env",
+          {
+            "modules": false
+          }
+        ],
+        "@babel/preset-react"
+      ],
+      "plugins": [
+      ]
+    }
+  }
+}

src/util.js src/util/index.jssrc/util.js renamed to src/util/index.js

@@ -2410,6 +2410,11 @@ babel-plugin-transform-react-jsx@6.24.1, babel-plugin-transform-react-jsx@^6.24.
     babel-plugin-syntax-jsx "^6.8.0"
     babel-runtime "^6.22.0"
 
+babel-plugin-transform-react-remove-prop-types@^0.4.19:
+  version "0.4.19"
+  resolved "https://registry.yarnpkg.com/babel-plugin-transform-react-remove-prop-types/-/babel-plugin-transform-react-remove-prop-types-0.4.19.tgz#dc9d8fb176a407a75efe73f231550450e29a3b17"
+  integrity sha512-f49NsaohQ1ByY20nUrpc30QFdbeT4ntV4PAL2vSZe6uCB5nqAcqXS/qzU+aI6ZfYhWASx5eIsTFvFrs1B2ffGg==
+
 babel-plugin-transform-regenerator@6.26.0, babel-plugin-transform-regenerator@^6.22.0, babel-plugin-transform-regenerator@^6.26.0, babel-plugin-transform-regenerator@^6.5.0:
   version "6.26.0"
   resolved "https://registry.yarnpkg.com/babel-plugin-transform-regenerator/-/babel-plugin-transform-regenerator-6.26.0.tgz#e0703696fbde27f0a3efcacf8b4dca2f7b3a8f2f"