From 43b9d95dbc8d645564bd82a4f264d9d3cfb47270 Mon Sep 17 00:00:00 2001
From: PixelyIon <pixelyion@protonmail.com>
Date: Thu, 11 Nov 2021 13:47:20 +0530
Subject: [PATCH] Rework App Dialog Buttons

Aligning the buttons to the bottom of the game image was determined to look odd due to the amount of padding between the title and buttons. They are now back to being below the title but the buttons have been resized with "Play" being a wide button while "Pin" has been replaced with Google Material Icons's "Add To Home Screen" icon and sized down to an icon-only button.
---
 app/src/main/res/drawable/ic_add_home.xml | 11 +++++++++++
 app/src/main/res/layout/app_dialog.xml    | 12 ++++++++----
 2 files changed, 19 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/res/drawable/ic_add_home.xml

diff --git a/app/src/main/res/drawable/ic_add_home.xml b/app/src/main/res/drawable/ic_add_home.xml
new file mode 100644
index 00000000..ca546a67
--- /dev/null
+++ b/app/src/main/res/drawable/ic_add_home.xml
@@ -0,0 +1,11 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="24"
+    android:viewportHeight="24"
+    android:tint="?attr/colorControlNormal"
+    android:autoMirrored="true">
+  <path
+      android:fillColor="@android:color/white"
+      android:pathData="M18,1.01L8,1c-1.1,0 -2,0.9 -2,2v3h2V5h10v14H8v-1H6v3c0,1.1 0.9,2 2,2h10c1.1,0 2,-0.9 2,-2V3c0,-1.1 -0.9,-1.99 -2,-1.99zM10,15h2V8H5v2h3.59L3,15.59 4.41,17 10,11.41z"/>
+</vector>
diff --git a/app/src/main/res/layout/app_dialog.xml b/app/src/main/res/layout/app_dialog.xml
index babc8239..98a530f2 100644
--- a/app/src/main/res/layout/app_dialog.xml
+++ b/app/src/main/res/layout/app_dialog.xml
@@ -45,12 +45,12 @@
             tools:text="Nintendo" />
 
         <com.google.android.flexbox.FlexboxLayout
-            android:layout_width="wrap_content"
+            android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_gravity="start"
-            android:layout_marginTop="8dp"
+            android:layout_marginTop="12dp"
             app:layout_constraintStart_toStartOf="@id/game_title"
-            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/game_subtitle"
             app:flexWrap="wrap">
 
             <Button
@@ -62,6 +62,7 @@
                 android:focusedByDefault="true"
                 android:text="@string/play"
                 android:textColor="?attr/colorAccent"
+                app:layout_minWidth="146dp"
                 app:icon="@drawable/ic_play"
                 app:iconTint="?attr/colorAccent" />
 
@@ -71,7 +72,10 @@
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_marginStart="6dp"
-                android:text="@string/pin"
+                app:layout_maxWidth="55dp"
+                app:iconGravity="textStart"
+                app:iconPadding="0dp"
+                app:icon="@drawable/ic_add_home"
                 android:textColor="?attr/colorAccent" />
         </com.google.android.flexbox.FlexboxLayout>
     </androidx.constraintlayout.widget.ConstraintLayout>