add logger to package.json and remove duplicate log

facebook · Sep 27, 2017 · b02372d · b02372d
1 parent b7e45f2
commit b02372d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/packages/react-dev-utils/logger.js b/packages/react-dev-utils/logger.js
@@ -3,14 +3,13 @@ const React = require('react');
 const ReactDOM = require('react-dom');
 const h = React.createElement;
 
-const useLogger = true;
-const domId = 'toast';
+const id = 'create-react-app-logger';
 
-let dom = document.getElementById(domId);
+let dom = document.getElementById(id);
 
 if (!dom) {
   dom = document.createElement('div');
-  dom.id = domId;
+  dom.id = id;
   document.body.appendChild(dom);
 }
 
@@ -82,9 +81,11 @@ class Logger extends React.Component {
       });
     }, this.props.timeout);
   }
+
   componentWillUnmount() {
     clearTimeout(this.timeout);
   }
+
   render() {
     return h(
       'div',
@@ -128,11 +129,7 @@ function getColor(type) {
 }
 
 function render(type, message) {
-  if (useLogger) {
-    ReactDOM.render(h(Logger, { type }, message), dom);
-  } else {
-    console.log(message);
-  }
+  ReactDOM.render(h(Logger, { type }, message), dom);
 }
 
 const logger = {

diff --git a/packages/react-dev-utils/package.json b/packages/react-dev-utils/package.json
@@ -24,7 +24,8 @@
     "inquirer.js",
     "InterpolateHtmlPlugin.js",
     "launchEditor.js",
-    "launchEditorEndpoint.js",
+    "logger.js",
+    "noopServiceWorkerMiddleware.js",
     "ModuleScopePlugin.js",
     "noopServiceWorkerMiddleware.js",
     "openBrowser.js",