From 465361bf0521f6854600c7e8e911f278949110f1 Mon Sep 17 00:00:00 2001
From: maciejhirsz <maciej.hirsz@gmail.com>
Date: Thu, 19 Jul 2018 13:41:05 +0200
Subject: [PATCH] Only render node details on map when hovered

---
 packages/frontend/src/components/Node.css |  5 --
 packages/frontend/src/components/Node.tsx | 61 ++++++++++++++++-------
 2 files changed, 44 insertions(+), 22 deletions(-)

diff --git a/packages/frontend/src/components/Node.css b/packages/frontend/src/components/Node.css
index 7183650..50064a9 100644
--- a/packages/frontend/src/components/Node.css
+++ b/packages/frontend/src/components/Node.css
@@ -29,7 +29,6 @@
 }
 
 .Node-details {
-  display: none;
   min-width: 335px;
   position: absolute;
   left: 16px;
@@ -55,7 +54,3 @@
 .Node-details td:nth-child(even) {
   padding-left: 0.2em;
 }
-
-.Node-Location:hover .Node-details {
-  display: initial;
-}
diff --git a/packages/frontend/src/components/Node.tsx b/packages/frontend/src/components/Node.tsx
index fe7bb92..6fac294 100644
--- a/packages/frontend/src/components/Node.tsx
+++ b/packages/frontend/src/components/Node.tsx
@@ -30,6 +30,10 @@ export namespace Node {
     top: number;
   }
 
+  export interface LocationState {
+    hover: boolean;
+  }
+
   export function Header() {
     return (
       <thead>
@@ -68,25 +72,40 @@ export namespace Node {
     );
   }
 
-  export function Location(props: Props & PixelPosition) {
-    const { left, top, location } = props;
-    const [name, implementation, version] = props.nodeDetails;
-    const [height, hash, blockTime, blockTimestamp, propagationTime] = props.blockDetails;
+  export class Location extends React.Component<Props & PixelPosition, LocationState> {
+    public readonly state = { hover: false };
 
-    if (!location) {
-      return null;
+    public render() {
+      const { left, top, location } = this.props;
+      const height = this.props.blockDetails[0];
+      const propagationTime = this.props.blockDetails[4];
+
+      if (!location) {
+        return null;
+      }
+
+      let className = 'Node-Location';
+
+      if (propagationTime) {
+        className += ' Node-Location-synced';
+      } else if (height % 2 === 1) {
+        className += ' Node-Location-odd';
+      }
+
+      return (
+        <div className={className} style={{ left, top }} onMouseOver={this.onMouseOver} onMouseOut={this.onMouseOut}>
+        {
+          this.state.hover ? this.renderDetails(location) : null
+        }
+        </div>
+      );
     }
 
-    let className = 'Node-Location';
+    private renderDetails(location: Types.NodeLocation) {
+      const [name, implementation, version] = this.props.nodeDetails;
+      const [height, hash, blockTime, blockTimestamp, propagationTime] = this.props.blockDetails;
 
-    if (propagationTime) {
-      className += ' Node-Location-synced';
-    } else if (height % 2 === 1) {
-      className += ' Node-Location-odd';
-    }
-
-    return (
-      <div className={className} style={{ left, top }}>
+      return (
         <table className="Node-details">
           <tbody>
             <tr>
@@ -114,7 +133,15 @@ export namespace Node {
             </tr>
           </tbody>
         </table>
-      </div>
-    );
+      );
+    }
+
+    private onMouseOver = () => {
+      this.setState({ hover: true });
+    }
+
+    private onMouseOut = () => {
+      this.setState({ hover: false });
+    }
   }
 }