Merge pull request #135 from weaveworks/document-title

Set document title for node details

client/app/scripts/components/node-details.js

@@ -2,6 +2,7 @@ const React = require('react');
 
 const NodeDetailsTable = require('./node-details-table');
 const NodeColorMixin = require('../mixins/node-color-mixin');
+const TitleUtils = require('../utils/title-utils');
 
 const NodeDetails = React.createClass({
 
@@ -9,6 +10,14 @@ const NodeDetails = React.createClass({
     NodeColorMixin
   ],
 
+  componentDidMount: function() {
+    this.updateTitle();
+  },
+
+  componentWillUnmount: function() {
+    TitleUtils.resetTitle();
+  },
+
   render: function() {
     const node = this.props.details;
 
@@ -36,6 +45,14 @@ const NodeDetails = React.createClass({
         </div>
       </div>
     );
+  },
+
+  componentDidUpdate: function() {
+    this.updateTitle();
+  },
+
+  updateTitle: function() {
+    TitleUtils.setTitle(this.props.details && this.props.details.label_major);
   }
 
 });

client/app/scripts/utils/title-utils.js

@@ -0,0 +1,20 @@
+
+const PREFIX = 'Weave Scope';
+const SEPARATOR = ' - ';
+
+function setDocumentTitle(title) {
+  if (title) {
+    document.title = [PREFIX, title].join(SEPARATOR);
+  } else {
+    document.title = PREFIX;
+  }
+}
+
+function resetDocumentTitle() {
+  setDocumentTitle(null);
+}
+
+module.exports = {
+  resetTitle: resetDocumentTitle,
+  setTitle: setDocumentTitle
+};