From d3661166d8ba8a7c904c4aa472875d97566b7549 Mon Sep 17 00:00:00 2001
From: Cristian Monforte <cristian@duckduckgo.com>
Date: Wed, 1 Apr 2020 11:31:47 +0200
Subject: [PATCH 1/2] UI fixes for tablets.

- changing alignment and max-width for dax dialogs
---
 .../main/res/layout-land/content_onboarding_welcome.xml   | 7 ++++---
 app/src/main/res/layout/content_dax_dialog.xml            | 7 ++++---
 app/src/main/res/layout/content_onboarding_welcome.xml    | 7 ++++---
 app/src/main/res/layout/include_new_browser_tab.xml       | 8 +++++---
 4 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/app/src/main/res/layout-land/content_onboarding_welcome.xml b/app/src/main/res/layout-land/content_onboarding_welcome.xml
index a6efef3d3f94..ac8c15dcd4aa 100644
--- a/app/src/main/res/layout-land/content_onboarding_welcome.xml
+++ b/app/src/main/res/layout-land/content_onboarding_welcome.xml
@@ -59,11 +59,12 @@
 
     <include
         layout="@layout/include_dax_dialog_cta"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
         android:layout_marginTop="93dp"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
+        app:layout_constraintTop_toTopOf="parent"
+        app:layout_constraintWidth_max="600dp"/>
 
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/content_dax_dialog.xml b/app/src/main/res/layout/content_dax_dialog.xml
index 38a39cf567a5..48c154312bac 100644
--- a/app/src/main/res/layout/content_dax_dialog.xml
+++ b/app/src/main/res/layout/content_dax_dialog.xml
@@ -60,7 +60,7 @@
             android:layout_marginStart="24dp"
             android:layout_marginBottom="16dp"
             app:layout_constraintBottom_toTopOf="@id/cardView"
-            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintStart_toStartOf="@id/cardView"
             app:srcCompat="@drawable/ic_dax_icon" />
 
         <androidx.appcompat.widget.AppCompatImageView
@@ -74,7 +74,7 @@
 
         <androidx.cardview.widget.CardView
             android:id="@+id/cardView"
-            android:layout_width="match_parent"
+            android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:layout_marginStart="8dp"
             android:layout_marginEnd="8dp"
@@ -83,7 +83,8 @@
             app:cardCornerRadius="20dp"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
-            app:layout_constraintStart_toStartOf="parent">
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintWidth_max="600dp">
 
             <LinearLayout
                 android:id="@+id/cardContainer"
diff --git a/app/src/main/res/layout/content_onboarding_welcome.xml b/app/src/main/res/layout/content_onboarding_welcome.xml
index fa482e1902ca..4a8e2e64d9f9 100644
--- a/app/src/main/res/layout/content_onboarding_welcome.xml
+++ b/app/src/main/res/layout/content_onboarding_welcome.xml
@@ -59,11 +59,12 @@
 
     <include
         layout="@layout/include_dax_dialog_cta"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
         android:layout_marginTop="93dp"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
+        app:layout_constraintTop_toTopOf="parent"
+        app:layout_constraintWidth_max="600dp"/>
 
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/include_new_browser_tab.xml b/app/src/main/res/layout/include_new_browser_tab.xml
index 6af6405cafc6..ab436f4fab1f 100644
--- a/app/src/main/res/layout/include_new_browser_tab.xml
+++ b/app/src/main/res/layout/include_new_browser_tab.xml
@@ -54,12 +54,14 @@
 
     <include
         layout="@layout/include_dax_dialog_cta"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
         android:layout_marginTop="40dp"
+        app:layout_constraintHorizontal_bias="0"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
+        app:layout_constraintTop_toTopOf="parent"
+        app:layout_constraintWidth_max="600dp"/>
 
     <LinearLayout
         android:id="@+id/ctaContainer"

From 7709432524af1875d1929b7658c4575bcccc9437 Mon Sep 17 00:00:00 2001
From: Cristian Monforte <cristian@duckduckgo.com>
Date: Wed, 1 Apr 2020 14:29:55 +0200
Subject: [PATCH 2/2] Showing centered also bubble dialogs

---
 app/src/main/res/layout/include_new_browser_tab.xml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/app/src/main/res/layout/include_new_browser_tab.xml b/app/src/main/res/layout/include_new_browser_tab.xml
index ab436f4fab1f..256ed3668e64 100644
--- a/app/src/main/res/layout/include_new_browser_tab.xml
+++ b/app/src/main/res/layout/include_new_browser_tab.xml
@@ -57,7 +57,6 @@
         android:layout_width="0dp"
         android:layout_height="wrap_content"
         android:layout_marginTop="40dp"
-        app:layout_constraintHorizontal_bias="0"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"