From 0b213d83e329c55f08b722af3e7e47ec9688fd4e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Magiera?= <magik6k@gmail.com>
Date: Wed, 31 Jul 2019 01:21:57 +0200
Subject: [PATCH] pond: Move window-related stuff to App.js

---
 lotuspond/front/src/App.js      | 23 +++++++++++++++++++++--
 lotuspond/front/src/NodeList.js | 22 ++--------------------
 2 files changed, 23 insertions(+), 22 deletions(-)

diff --git a/lotuspond/front/src/App.js b/lotuspond/front/src/App.js
index a5ead67ac..e2ea90732 100644
--- a/lotuspond/front/src/App.js
+++ b/lotuspond/front/src/App.js
@@ -13,7 +13,23 @@ class App extends React.Component {
       this.setState(() => ({client: client}))
     })
 
-    this.state = {}
+    this.state = {
+      windows: {},
+      nextWindow: 0,
+    }
+
+    this.mountWindow = this.mountWindow.bind(this)
+  }
+
+  mountWindow(cb) {
+    const id = this.state.nextWindow
+    this.setState({nextWindow: id + 1})
+
+    const window = cb(() => {
+      this.setState(prev => ({windows: {...prev.windows, [id]: undefined}}))
+    })
+
+    this.setState(prev => ({windows: {...prev.windows, [id]: window}}))
   }
 
   render() {
@@ -27,7 +43,10 @@ class App extends React.Component {
 
     return (
         <div className="App">
-          <NodeList client={this.state.client}/>
+          <NodeList client={this.state.client} mountWindow={this.mountWindow}/>
+          <div>
+            {Object.keys(this.state.windows).map((w, i) => <div key={i}>{this.state.windows[w]}</div>)}
+          </div>
         </div>
     )
   }
diff --git a/lotuspond/front/src/NodeList.js b/lotuspond/front/src/NodeList.js
index d92cfb279..75bddc342 100644
--- a/lotuspond/front/src/NodeList.js
+++ b/lotuspond/front/src/NodeList.js
@@ -12,16 +12,12 @@ class NodeList extends React.Component {
 
       showConnMgr: false,
       showConsensus: false,
-
-      windows: {},
-      nextWindow: 0,
     }
 
     // This binding is necessary to make `this` work in the callback
     this.spawnNode = this.spawnNode.bind(this)
     this.connMgr = this.connMgr.bind(this)
     this.consensus = this.consensus.bind(this)
-    this.mountWindow = this.mountWindow.bind(this)
 
     this.getNodes()
   }
@@ -47,17 +43,6 @@ class NodeList extends React.Component {
     this.setState({showConsensus: true})
   }
 
-  mountWindow(cb) {
-    const id = this.state.nextWindow
-    this.setState({nextWindow: id + 1})
-
-    const window = cb(() => {
-      this.setState(prev => ({windows: {...prev.windows, [id]: undefined}}))
-    })
-
-    this.setState(prev => ({windows: {...prev.windows, [id]: window}}))
-  }
-
   render() {
     let connMgr
     if (this.state.showConnMgr) {
@@ -66,7 +51,7 @@ class NodeList extends React.Component {
 
     let consensus
     if (this.state.showConsensus) {
-      consensus = (<Consensus nodes={this.state.nodes} mountWindow={this.mountWindow}/>)
+      consensus = (<Consensus nodes={this.state.nodes} mountWindow={this.props.mountWindow}/>)
     }
 
     return (
@@ -85,15 +70,12 @@ class NodeList extends React.Component {
                                 node={{...node}}
                                 pondClient={this.props.client}
                                 onConnect={(conn, id) => this.setState(prev => ({nodes: {...prev.nodes, [n]: {...node, conn: conn, peerid: id}}}))}
-                                mountWindow={this.mountWindow}/>)
+                                mountWindow={this.props.mountWindow}/>)
             })
           }
           {connMgr}
           {consensus}
         </div>
-        <div>
-          {Object.keys(this.state.windows).map((w, i) => <div key={i}>{this.state.windows[w]}</div>)}
-        </div>
       </div>
     );
   }