DD: Pull up button, error text, ViewModel

ListView should really only be a list.
Makes more sense to add error after rfresh button.

1 files changed, 26 insertions, 41 deletions

diff --git a/lib/widgets/badge_motive_list.dart b/lib/widgets/badge_motive_list.dart
index a03e174..989b155 100644
--- a/lib/widgets/badge_motive_list.dart
+++ b/lib/widgets/badge_motive_list.dart
@@ -41,48 +41,33 @@ class _BadgeMotiveListState extends State<BadgeMotiveList> {
   Widget build(BuildContext context) {
     var mytheme = Theme.of(context);
 
-    return Flexible(
-      fit: FlexFit.loose,
-      child: Column(
-        children: [
-          ElevatedButton(
-            child: Text("Refresh"),
-            onPressed: () async {
-              await widget._motiveVM.updateMotives();
-              await widget._motiveVM.getCurrentMotive();
+    return Expanded(
+      child: ListenableBuilder(
+        listenable: widget._motiveVM,
+        builder: (context, child) {
+          return ListView.separated(
+            itemBuilder: (context, index) {
+              final item = widget._motiveVM.motives[index];
+              final selected = widget._motiveVM.currentMotive?.id == item.id;
+              return ListTile(
+                title: Text(item.description),
+                selectedTileColor: mytheme.primaryColorLight,
+                selectedColor: Colors.black,
+                onTap: !widget._motiveVM.allowSelection
+                    ? null
+                    : () async {
+                        widget.onItemSelected(item);
+                        await widget._motiveVM.setMotive(item);
+                      },
+                selected: selected,
+              );
             },
-          ),
-          Expanded(
-            child: ListenableBuilder(
-              listenable: widget._motiveVM,
-              builder: (context, child) {
-                return ListView.separated(
-                  itemBuilder: (context, index) {
-                    final item = widget._motiveVM.motives[index];
-                    final selected =
-                        widget._motiveVM.currentMotive?.id == item.id;
-                    return ListTile(
-                      title: Text(item.description),
-                      selectedTileColor: mytheme.primaryColorLight,
-                      selectedColor: Colors.black,
-                      onTap: !widget._motiveVM.allowSelection
-                          ? null
-                          : () async {
-                              widget.onItemSelected(item);
-                              await widget._motiveVM.setMotive(item);
-                            },
-                      selected: selected,
-                    );
-                  },
-                  separatorBuilder: (context, index) {
-                    return Divider();
-                  },
-                  itemCount: widget._motiveVM.motives.length,
-                );
-              },
-            ),
-          ),
-        ],
+            separatorBuilder: (context, index) {
+              return Divider();
+            },
+            itemCount: widget._motiveVM.motives.length,
+          );
+        },
       ),
     );
   }