From 5f4008b6754cd81bca6339b896536d60db349f31 Mon Sep 17 00:00:00 2001
From: John Furrow <johnf@mesosphere.io>
Date: Sat, 10 Dec 2016 21:04:36 -0800
Subject: [PATCH] Rearrange methods

---
 .../components/Sidebar/TransferRateGraph.js   | 122 ++++++++++--------
 1 file changed, 65 insertions(+), 57 deletions(-)

diff --git a/client/scripts/components/Sidebar/TransferRateGraph.js b/client/scripts/components/Sidebar/TransferRateGraph.js
index 5224f222..87113b41 100644
--- a/client/scripts/components/Sidebar/TransferRateGraph.js
+++ b/client/scripts/components/Sidebar/TransferRateGraph.js
@@ -48,37 +48,12 @@ class LineChart extends React.Component {
     });
   }
 
+  componentDidMount() {
+    this.updateGraph();
+  }
+
   componentDidUpdate() {
-    this.renderGraphData(this.props);
-
-    if (this.graphRefs.isHovered) {
-      this.renderPrecisePointInspectors();
-    }
-  }
-
-  handleMouseOut() {
-    const {graphRefs, props} = this;
-
-    if (graphRefs.areDefined) {
-      graphRefs.isHovered = false;
-      graphRefs.upload.inspectPoint.style('opacity', 0);
-      graphRefs.download.inspectPoint.style('opacity', 0);
-    }
-
-    if (props.onMouseOut) {
-      props.onMouseOut();
-    }
-  }
-
-  handleMouseMove(mouseX) {
-    this.lastMouseX = mouseX;
-    this.renderPrecisePointInspectors();
-  }
-
-  handleMouseOver() {
-    this.graphRefs.isHovered = true;
-    this.graphRefs.upload.inspectPoint.style('opacity', 1);
-    this.graphRefs.download.inspectPoint.style('opacity', 1);
+    this.updateGraph();
   }
 
   appendGraphCircles(graph, slug) {
@@ -105,6 +80,41 @@ class LineChart extends React.Component {
       .attr('class', `graph__line graph__line--${slug}`);
   }
 
+  getGradient(slug) {
+    return (
+      <linearGradient id={`graph__gradient--${slug}`} x1="0%" y1="0%"
+        x2="0%" y2="100%">
+        <stop className={`graph__gradient--top graph__gradient--top--${slug}`} offset="0%"/>
+        <stop className={`graph__gradient--bottom graph__gradient--bottom--${slug}`} offset="100%"/>
+      </linearGradient>
+    );
+  }
+
+  handleMouseMove(mouseX) {
+    this.lastMouseX = mouseX;
+    this.renderPrecisePointInspectors();
+  }
+
+  handleMouseOut() {
+    const {graphRefs, props} = this;
+
+    if (graphRefs.areDefined) {
+      graphRefs.isHovered = false;
+      graphRefs.upload.inspectPoint.style('opacity', 0);
+      graphRefs.download.inspectPoint.style('opacity', 0);
+    }
+
+    if (props.onMouseOut) {
+      props.onMouseOut();
+    }
+  }
+
+  handleMouseOver() {
+    this.graphRefs.isHovered = true;
+    this.graphRefs.upload.inspectPoint.style('opacity', 1);
+    this.graphRefs.download.inspectPoint.style('opacity', 1);
+  }
+
   renderGraphData(props) {
     const {height, historicalData, id, width} = props;
     const graph = d3.select(`#${id}`);
@@ -166,6 +176,27 @@ class LineChart extends React.Component {
     this.graphRefs.upload.rateLine.attr('d', uploadLinePath);
   }
 
+  renderPrecisePointInspectors() {
+    const {
+      lastMouseX,
+      props: {historicalData, onHover},
+      xScale
+    } = this;
+
+    const hoverPoint = xScale.invert(lastMouseX);
+    const uploadSpeed = this.setInspectorCoordinates('upload', hoverPoint);
+    const downloadSpeed = this.setInspectorCoordinates('download', hoverPoint);
+    const nearestTimestamp = historicalData.timestamps[Math.round(hoverPoint)];
+
+    if (onHover) {
+      onHover({
+        uploadSpeed,
+        downloadSpeed,
+        nearestTimestamp
+      });
+    }
+  }
+
   setInspectorCoordinates(slug, hoverPoint) {
     const {
       graphRefs: {[slug]: {inspectPoint}},
@@ -189,37 +220,14 @@ class LineChart extends React.Component {
     return speedAtHoverPoint;
   }
 
-  renderPrecisePointInspectors() {
-    const {
-      lastMouseX,
-      props: {historicalData, onHover},
-      xScale
-    } = this;
+  updateGraph() {
+    this.renderGraphData(this.props);
 
-    const hoverPoint = xScale.invert(lastMouseX);
-    const uploadSpeed = this.setInspectorCoordinates('upload', hoverPoint);
-    const downloadSpeed = this.setInspectorCoordinates('download', hoverPoint);
-    const nearestTimestamp = historicalData.timestamps[Math.round(hoverPoint)];
-
-    if (onHover) {
-      onHover({
-        uploadSpeed,
-        downloadSpeed,
-        nearestTimestamp
-      });
+    if (this.graphRefs.isHovered) {
+      this.renderPrecisePointInspectors();
     }
   }
 
-  getGradient(slug) {
-    return (
-      <linearGradient id={`graph__gradient--${slug}`} x1="0%" y1="0%"
-        x2="0%" y2="100%">
-        <stop className={`graph__gradient--top graph__gradient--top--${slug}`} offset="0%"/>
-        <stop className={`graph__gradient--bottom graph__gradient--bottom--${slug}`} offset="100%"/>
-      </linearGradient>
-    );
-  }
-
   render() {
     return (
       <svg className="graph" id={this.props.id}