fixed roll builder state

DungeonPaper · Jun 23, 2020 · 033bd3d · 033bd3d
1 parent 888fbd1
commit 033bd3d
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 8 deletions.
diff --git a/lib/src/atoms/modifier_selector.dart b/lib/src/atoms/modifier_selector.dart
@@ -31,7 +31,7 @@ class ModifierSelector extends StatelessWidget {
         color: Theme.of(context).colorScheme.onSurface.withOpacity(0.4),
         fontSize: normalText.fontSize * 0.75);
     return Container(
-      width: 100,
+      width: 70,
       child: DropdownButton(
         isExpanded: true,
         value: value,

diff --git a/lib/src/dialogs/roll_dice_dialog.dart b/lib/src/dialogs/roll_dice_dialog.dart
@@ -19,15 +19,13 @@ class RollDiceDialog extends StatefulWidget {
 }
 
 class _RollDiceDialogState extends State<RollDiceDialog> {
-  DiceListController addingController;
-  List<List<Dice>> diceList = [];
-  List<DiceListController> controllers = [];
-
-  static List<Dice> DEFAULT_DICE = [Dice.d6 * 2];
+  List<List<Dice>> diceList;
+  List<DiceListController> controllers;
 
   @override
   void initState() {
-    addingController = DiceListController(DEFAULT_DICE);
+    diceList = [];
+    controllers = [];
     super.initState();
   }
 

diff --git a/lib/src/molecules/dice_roll_box.dart b/lib/src/molecules/dice_roll_box.dart
@@ -68,6 +68,9 @@ class _DiceRollBoxState extends State<DiceRollBox>
       padding: const EdgeInsets.symmetric(horizontal: 16),
       child: Card(
         color: Theme.of(context).canvasColor,
+        shape: RoundedRectangleBorder(
+          borderRadius: BorderRadius.all(Radius.circular(10)),
+        ),
         child: Column(
           children: [
             Container(

diff --git a/lib/src/molecules/dice_roll_builder.dart b/lib/src/molecules/dice_roll_builder.dart
@@ -29,7 +29,7 @@ class _DiceRollBuilderState extends State<DiceRollBuilder> {
 
   @override
   void initState() {
-    addingController = DiceListController(DEFAULT_DICE);
+    addingController = DiceListController(List.from(DEFAULT_DICE));
     super.initState();
   }
 
@@ -38,6 +38,9 @@ class _DiceRollBuilderState extends State<DiceRollBuilder> {
     return Card(
       color: Theme.of(context).canvasColor,
       elevation: 3,
+      shape: RoundedRectangleBorder(
+        borderRadius: BorderRadius.all(Radius.circular(10)),
+      ),
       child: Column(
         crossAxisAlignment: CrossAxisAlignment.stretch,
         children: [
@@ -68,12 +71,14 @@ class _DiceRollBuilderState extends State<DiceRollBuilder> {
                           }
                         },
                       ),
+                      SizedBox(width: 16),
                       ModifierSelector(
                         value: d.value.modifier,
                         character: widget.character,
                         onChanged: (n) => _setMod(d.index, n),
                         textStyle: TextStyle(fontSize: 20),
                       ),
+                      Expanded(child: Container()),
                       if (addingController.value.length > 1)
                         IconButton(
                           icon: Icon(Icons.close),