From: Kevin Worth <kworth082@gmail.com>
Date: Tue, 12 May 2020 03:16:20 +0000 (-0400)
Subject: Get rid of extra FutureBuilder widgets
X-Git-Url: https://git.cworth.org/git?p=lmno.games;a=commitdiff_plain;h=aaf309c2de34f2381a9b48ad404c89bc7d43780d;hp=8052a68188149faed5405180b20245d8b686b07a

Get rid of extra FutureBuilder widgets

Now that the ListView is working correctly, we can get rid of these two
FutureBuilders which were essentially intermediate steps to get to this
point.
---

diff --git a/flutterempires/lib/main.dart b/flutterempires/lib/main.dart
index 89867ec..64d8bf0 100644
--- a/flutterempires/lib/main.dart
+++ b/flutterempires/lib/main.dart
@@ -110,32 +110,8 @@ class _MyHomePageState extends State<MyHomePage> {
                 ),
               ),
               Spacer(flex: 1),
-              FutureBuilder<Player>(
-                future: futurePlayer,
-                builder: (context, snapshot) {
-                  if (snapshot.hasData) {
-                    return Text(snapshot.data.name);
-                  } else if (snapshot.hasError) {
-                    return Text("${snapshot.error}");
-                  }
-                  return CircularProgressIndicator();
-                },
-              ),
-              Spacer(flex: 1),
-              FutureBuilder<List<Player>>(
-                future: allPlayers,
-                builder: (context, snapshot) {
-                  if (snapshot.hasData) {
-                    return Text(snapshot.data.length.toString());
-                  } else if (snapshot.hasError) {
-                    return Text("${snapshot.error}");
-                  }
-                  // By default, show a loading spinner.
-                  return CircularProgressIndicator();
-                },
-              ),
               Expanded(
-                flex: 10,
+                flex: 20,
                 child: FutureBuilder<List<Player>>(
                     future: allPlayers,
                     builder: (context, snapshot) {