From 584dc2906ede9ec4b39a7870b5875154b0968ed3 Mon Sep 17 00:00:00 2001
From: Kevin Worth <kworth082@gmail.com>
Date: Sun, 3 May 2020 15:31:41 -0400
Subject: [PATCH] Add a FutureBuilder widget to hold a fetched player

Rather than doing a fetch within this build method (which gets called repeatedly),
we use a FutureBuilder to show a spinner until the player has finished being
fetched asynchronously.
---
 flutterempires/lib/main.dart | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/flutterempires/lib/main.dart b/flutterempires/lib/main.dart
index 92cda02..7bb1c7d 100644
--- a/flutterempires/lib/main.dart
+++ b/flutterempires/lib/main.dart
@@ -1,4 +1,5 @@
 import 'package:flutter/material.dart';
+import 'package:flutterempires/player.dart';
 
 void main() {
   runApp(MyApp());
@@ -94,7 +95,7 @@ class _MyHomePageState extends State<MyHomePage> {
                   hintText: 'Enter your (real) name',
                 ),
               ),
-              Spacer(),
+              Spacer(flex: 1),
               Text(
                 'Character:',
                 style: Theme.of(context).textTheme.headline4,
@@ -104,6 +105,13 @@ class _MyHomePageState extends State<MyHomePage> {
                   hintText: 'Enter your empire character name',
                 ),
               ),
+              Spacer(flex: 1),
+              FutureBuilder<Player>(
+                builder: (context, snapshot) {
+                  // By default, show a loading spinner.
+                  return CircularProgressIndicator();
+                },
+              ),
               Spacer(flex: 10),
             ],
           ),
-- 
2.43.0