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

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) {