Don't use a nested component

gaearon · Feb 16, 2015 · b4f32c8 · b4f32c8
1 parent a4f6ac8
commit b4f32c8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/index.js b/index.js
@@ -12,17 +12,15 @@ function getTitleFromPropsList(propsList) {
   }
 }
 
-var SetDocumentTitle = createSideEffect(function handleChange(propsList) {
+var DocumentTitle = createSideEffect(function handleChange(propsList) {
   var title = getTitleFromPropsList(propsList);
 
   if (typeof document !== 'undefined') {
     document.title = title || '';
   } else {
     _serverTitle = title || null;
   }
-});
-
-var DocumentTitle = React.createClass({
+}, {
   displayName: 'DocumentTitle',
 
   propTypes: {
@@ -36,13 +34,9 @@ var DocumentTitle = React.createClass({
 
     rewind: function () {
       var title = _serverTitle;
-      SetDocumentTitle.dispose();
+      this.dispose();
       return title;
     }
-  },
-
-  render: function () {
-    return React.createElement(SetDocumentTitle, this.props);
   }
 });
 

diff --git a/package.json b/package.json
@@ -36,6 +36,6 @@
     "react": "^0.12.0"
   },
   "dependencies": {
-    "react-side-effect": "~0.1.0"
+    "react-side-effect": "~0.2.0"
   }
 }