From 053a64063161c65a38c7c4739197d7ad8d3d9c99 Mon Sep 17 00:00:00 2001
From: Tom Hicks <headunter3@gmail.com>
Date: Thu, 1 Jul 2021 02:21:41 -0700
Subject: [PATCH] Makes radio buttons suck less. Styles the advantage picker
 and proficiency picker. Makes the saving throws screen nicer.

---
 app/src/main/res/color/radio_button_text.xml  |   5 +
 .../res/drawable/radio_button_checked.xml     |   6 +
 .../res/drawable/radio_button_selector.xml    |   5 +
 .../res/drawable/radio_button_unchecked.xml   |   6 +
 .../res/layout/component_advantage_picker.xml |  40 +++-
 .../layout/component_advantage_picker2.xml    |  78 +++++++
 .../layout/component_proficiency_picker.xml   |  40 +++-
 .../layout/fragment_edit_saving_throws.xml    | 196 ++++++++++++------
 app/src/main/res/values/colors.xml            |  17 +-
 app/src/main/res/values/dimens.xml            |   8 +-
 10 files changed, 315 insertions(+), 86 deletions(-)
 create mode 100644 app/src/main/res/color/radio_button_text.xml
 create mode 100644 app/src/main/res/drawable/radio_button_checked.xml
 create mode 100644 app/src/main/res/drawable/radio_button_selector.xml
 create mode 100644 app/src/main/res/drawable/radio_button_unchecked.xml
 create mode 100644 app/src/main/res/layout/component_advantage_picker2.xml

diff --git a/app/src/main/res/color/radio_button_text.xml b/app/src/main/res/color/radio_button_text.xml
new file mode 100644
index 0000000..2b573b3
--- /dev/null
+++ b/app/src/main/res/color/radio_button_text.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+    <item android:color="@color/colorOnPrimary" android:state_checked="true"/>
+    <item android:color="@color/colorPrimary" android:state_checked="false"/>
+</selector>
diff --git a/app/src/main/res/drawable/radio_button_checked.xml b/app/src/main/res/drawable/radio_button_checked.xml
new file mode 100644
index 0000000..d5548cd
--- /dev/null
+++ b/app/src/main/res/drawable/radio_button_checked.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle">
+    <solid android:color="@color/colorPrimary" />
+    <corners android:radius="@dimen/corner_radius" />
+</shape>
diff --git a/app/src/main/res/drawable/radio_button_selector.xml b/app/src/main/res/drawable/radio_button_selector.xml
new file mode 100644
index 0000000..b9e7998
--- /dev/null
+++ b/app/src/main/res/drawable/radio_button_selector.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+    <item android:drawable="@drawable/radio_button_checked" android:state_checked="true" />
+    <item android:drawable="@drawable/radio_button_unchecked" android:state_checked="false" />
+</selector>
diff --git a/app/src/main/res/drawable/radio_button_unchecked.xml b/app/src/main/res/drawable/radio_button_unchecked.xml
new file mode 100644
index 0000000..dd01b1b
--- /dev/null
+++ b/app/src/main/res/drawable/radio_button_unchecked.xml
@@ -0,0 +1,6 @@
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle">
+    <solid android:color="@android:color/background_light" />
+    <corners android:radius="@dimen/corner_radius" />
+    <stroke android:color="@color/colorPrimary" android:width="1dp"/>
+</shape>
diff --git a/app/src/main/res/layout/component_advantage_picker.xml b/app/src/main/res/layout/component_advantage_picker.xml
index 55473c4..103a73a 100644
--- a/app/src/main/res/layout/component_advantage_picker.xml
+++ b/app/src/main/res/layout/component_advantage_picker.xml
@@ -6,16 +6,17 @@
     android:layout_height="wrap_content"
     android:layout_margin="@dimen/text_margin"
     tools:context=".ui.components.AdvantagePicker">
-    <!-- // TODO: style this control to look less awful by default -->
 
     <TextView
         android:id="@+id/label"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
+        android:layout_marginHorizontal="@dimen/padding_normal"
+        android:layout_marginTop="@dimen/padding_small"
         android:text="@string/label_advantage"
+        android:textAppearance="@android:style/TextAppearance.Material.Medium"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent"
-        tools:text="Proficiency" />
+        app:layout_constraintTop_toTopOf="parent" />
 
     <RadioGroup
         android:id="@+id/group"
@@ -29,21 +30,48 @@
             android:id="@+id/hasNoAdvantage"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_advantage_none" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_advantage_none"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="true" />
 
         <com.google.android.material.radiobutton.MaterialRadioButton
             android:id="@+id/hasAdvantage"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_advantage_advantage" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_advantage_advantage"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
 
         <com.google.android.material.radiobutton.MaterialRadioButton
             android:id="@+id/hasDisadvantage"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_advantage_disadvantage" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_advantage_disadvantage"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
     </RadioGroup>
 </merge>
diff --git a/app/src/main/res/layout/component_advantage_picker2.xml b/app/src/main/res/layout/component_advantage_picker2.xml
new file mode 100644
index 0000000..1715eaa
--- /dev/null
+++ b/app/src/main/res/layout/component_advantage_picker2.xml
@@ -0,0 +1,78 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:layout_margin="@dimen/text_margin"
+    tools:context=".ui.components.AdvantagePicker">
+    <!-- // TODO: style this control to look less awful by default -->
+
+    <TextView
+        android:id="@+id/label"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginHorizontal="@dimen/padding_normal"
+        android:layout_marginTop="@dimen/padding_small"
+        android:text="@string/label_advantage"
+        android:textAppearance="@android:style/TextAppearance.Material.Body1"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+
+    <RadioGroup
+        android:id="@+id/group"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:orientation="horizontal"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintTop_toBottomOf="@id/label">
+
+        <com.google.android.material.radiobutton.MaterialRadioButton
+            android:id="@+id/hasNoAdvantage"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginHorizontal="4dp"
+            android:layout_marginVertical="8dp"
+            android:layout_weight="1"
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="8dp"
+            android:text="@string/label_advantage_none"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="true" />
+
+        <com.google.android.material.radiobutton.MaterialRadioButton
+            android:id="@+id/hasAdvantage"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginHorizontal="4dp"
+            android:layout_marginVertical="8dp"
+            android:layout_weight="1"
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="8dp"
+            android:text="@string/label_advantage_advantage"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
+
+        <com.google.android.material.radiobutton.MaterialRadioButton
+            android:id="@+id/hasDisadvantage"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginHorizontal="4dp"
+            android:layout_marginVertical="8dp"
+            android:layout_weight="1"
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="8dp"
+            android:text="@string/label_advantage_disadvantage"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
+    </RadioGroup>
+</androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/layout/component_proficiency_picker.xml b/app/src/main/res/layout/component_proficiency_picker.xml
index 390b27e..608f569 100644
--- a/app/src/main/res/layout/component_proficiency_picker.xml
+++ b/app/src/main/res/layout/component_proficiency_picker.xml
@@ -6,16 +6,17 @@
     android:layout_height="wrap_content"
     android:layout_margin="@dimen/text_margin"
     tools:context=".ui.components.ProficiencyPicker">
-    <!-- // TODO: style this control to look less awful by default -->
 
     <TextView
         android:id="@+id/label"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
+        android:layout_marginHorizontal="@dimen/padding_normal"
+        android:layout_marginTop="@dimen/padding_small"
         android:text="@string/label_proficiency"
+        android:textAppearance="@android:style/TextAppearance.Material.Medium"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent"
-        tools:text="Proficiency" />
+        app:layout_constraintTop_toTopOf="parent" />
 
     <RadioGroup
         android:id="@+id/group"
@@ -29,21 +30,48 @@
             android:id="@+id/none"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_proficiency_none" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_proficiency_none"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="true" />
 
         <com.google.android.material.radiobutton.MaterialRadioButton
             android:id="@+id/proficient"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_proficiency_proficient" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_proficiency_proficient"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
 
         <com.google.android.material.radiobutton.MaterialRadioButton
             android:id="@+id/expertise"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:layout_marginHorizontal="@dimen/padding_small"
+            android:layout_marginVertical="@dimen/padding_normal"
             android:layout_weight="1"
-            android:text="@string/label_proficiency_expertise" />
+            android:background="@drawable/radio_button_selector"
+            android:button="@android:color/transparent"
+            android:gravity="center"
+            android:padding="@dimen/padding_normal"
+            android:text="@string/label_proficiency_expertise"
+            android:textAppearance="@android:style/TextAppearance.Material.Button"
+            android:textColor="@color/radio_button_text"
+            tools:checked="false" />
     </RadioGroup>
 </merge>
diff --git a/app/src/main/res/layout/fragment_edit_saving_throws.xml b/app/src/main/res/layout/fragment_edit_saving_throws.xml
index 61b2c03..7d9df22 100644
--- a/app/src/main/res/layout/fragment_edit_saving_throws.xml
+++ b/app/src/main/res/layout/fragment_edit_saving_throws.xml
@@ -3,109 +3,169 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+
     tools:context=".ui.editmonster.EditSavingThrowsFragment">
 
     <LinearLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:orientation="vertical">
+        android:divider="?android:attr/dividerVertical"
+        android:orientation="vertical"
+        android:showDividers="middle">
 
-        <TextView
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_strength" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/strengthProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_strength"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/strengthAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/strengthProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
-        <TextView
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/strengthAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+        </LinearLayout>
+
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_dexterity" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/dexterityProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_dexterity"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/dexterityAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/dexterityProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
-        <TextView
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/dexterityAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+
+        </LinearLayout>
+
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_constitution" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/constitutionProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_constitution"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/constitutionAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/constitutionProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
-        <TextView
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/constitutionAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+
+        </LinearLayout>
+
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_intelligence" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/intelligenceProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_intelligence"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/intelligenceAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/intelligenceProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
-        <TextView
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/intelligenceAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+
+        </LinearLayout>
+
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_wisdom" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/wisdomProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_wisdom"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/wisdomAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/wisdomProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
-        <TextView
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/wisdomAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+
+        </LinearLayout>
+
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_gravity="start"
-            android:text="@string/label_charisma" />
+            android:layout_marginTop="@dimen/padding_large"
+            android:layout_marginBottom="@dimen/padding_normal"
+            android:orientation="vertical">
 
-        <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
-            android:id="@+id/charismaProficiency"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <TextView
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="center_horizontal"
+                android:text="@string/label_charisma"
+                android:textAppearance="@android:style/TextAppearance.Material.Title" />
 
-        <com.majinnaibu.monstercards.ui.components.AdvantagePicker
-            android:id="@+id/charismaAdvantage"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content" />
+            <com.majinnaibu.monstercards.ui.components.ProficiencyPicker
+                android:id="@+id/charismaProficiency"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
 
+            <com.majinnaibu.monstercards.ui.components.AdvantagePicker
+                android:id="@+id/charismaAdvantage"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content" />
+        </LinearLayout>
     </LinearLayout>
 
-</ScrollView>
\ No newline at end of file
+</ScrollView>
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index ba0ed7c..196b818 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -1,8 +1,17 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
-    <color name="colorPrimary">#9B2818</color>
-    <color name="colorPrimaryDark">#661A10</color>
-    <!--    <color name="colorAccent">#188B9B</color>-->
+    <!-- From palette here https://material.io/resources/color/#!/?view.left=1&view.right=1&primary.color=9b2818&secondary.color=ea7e00 -->
+    <color name="colorPrimary">#9b2818</color>
+    <color name="colorPrimaryLight">#d25941</color>
+    <color name="colorPrimaryDark">#660000</color>
+    <color name="colorSecondary">#ea7e00</color>
+    <color name="colorSecondaryLight">#ffae41</color>
+    <color name="colorSecondaryDark">#b15000</color>
+    <color name="colorOnPrimary">#ffffff</color>
+    <color name="colorOnSecondary">#000000</color>
+    <color name="colorOnLight">#ffffff</color>
+    <color name="colorOnDark">#000000</color>
+
     <color name="colorAccent">#995500</color>
 
     <!-- fruit system colors -->
@@ -14,4 +23,4 @@
     <color name="blue">#007AFF</color>
     <color name="purple">#5855D6</color>
     <color name="pink">#FF2D55</color>
-</resources>
\ No newline at end of file
+</resources>
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index c6c3b92..681f490 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -3,5 +3,9 @@
     <dimen name="text_margin">16dp</dimen>
     <dimen name="fab_margin">16dp</dimen>
     <dimen name="text_h4_size">20sp</dimen>
-
-</resources>
\ No newline at end of file
+    <dimen name="corner_radius">4dp</dimen>
+    <dimen name="padding_small">4dp</dimen>
+    <dimen name="padding_normal">8dp</dimen>
+    <dimen name="padding_large">16dp</dimen>
+    <dimen name="padding_huge">32dp</dimen>
+</resources>