From b86dd584dc6b1d2780e5f971d956773d93af2454 Mon Sep 17 00:00:00 2001
From: Daniel Gultsch <daniel@gultsch.de>
Date: Sat, 4 Mar 2017 15:07:25 +0100
Subject: [PATCH] made swipe handle more unified and smaller across various
 screen sizes

---
 src/main/res/layout/fragment_conversations_overview.xml | 3 ++-
 src/main/res/values-w300dp/dimens.xml                   | 3 ---
 src/main/res/values-w360dp/dimens.xml                   | 3 ---
 src/main/res/values-w384dp/dimens.xml                   | 4 +++-
 src/main/res/values-w585dp/dimens.xml                   | 2 +-
 src/main/res/values/dimens.xml                          | 2 +-
 6 files changed, 7 insertions(+), 10 deletions(-)
 delete mode 100644 src/main/res/values-w300dp/dimens.xml
 delete mode 100644 src/main/res/values-w360dp/dimens.xml

diff --git a/src/main/res/layout/fragment_conversations_overview.xml b/src/main/res/layout/fragment_conversations_overview.xml
index dcf9feb9562943f271cdb9b36700beb148d3a107..5c56166301a868b11fadbc5b0a48114295231292 100644
--- a/src/main/res/layout/fragment_conversations_overview.xml
+++ b/src/main/res/layout/fragment_conversations_overview.xml
@@ -5,7 +5,8 @@
 
     <LinearLayout
         xmlns:android="http://schemas.android.com/apk/res/android"
-        android:layout_width="@dimen/conversations_overview_width"
+        android:layout_width="match_parent"
+        android:layout_marginRight="@dimen/swipe_handle_size"
         android:layout_height="match_parent"
         android:background="?attr/color_background_primary"
         android:orientation="vertical" >
diff --git a/src/main/res/values-w300dp/dimens.xml b/src/main/res/values-w300dp/dimens.xml
deleted file mode 100644
index 28341a39591a5645a4e7f3b1a7dc10eb4b371490..0000000000000000000000000000000000000000
--- a/src/main/res/values-w300dp/dimens.xml
+++ /dev/null
@@ -1,3 +0,0 @@
-<resources>
-	<dimen name="conversations_overview_width">276dp</dimen>
-</resources>
diff --git a/src/main/res/values-w360dp/dimens.xml b/src/main/res/values-w360dp/dimens.xml
deleted file mode 100644
index 7e37ae16e9636e02269a2984ac7a256c1ee5ab8c..0000000000000000000000000000000000000000
--- a/src/main/res/values-w360dp/dimens.xml
+++ /dev/null
@@ -1,3 +0,0 @@
-<resources>
-	<dimen name="conversations_overview_width">304dp</dimen>
-</resources>
diff --git a/src/main/res/values-w384dp/dimens.xml b/src/main/res/values-w384dp/dimens.xml
index 441e1103fe38666ca25882e490c6b0e6fff8d6a4..cf58ceb30c1520613cfc0a0adf354b970bdf4da2 100644
--- a/src/main/res/values-w384dp/dimens.xml
+++ b/src/main/res/values-w384dp/dimens.xml
@@ -1,4 +1,6 @@
 <resources>
-	<dimen name="conversations_overview_width">336dp</dimen>
+	<!-- 384dp is the screen width of the Nexus 4. Something like a Moto G is smaller but a Nexus 5X is larger -->
+	<!-- https://material.io/devices/ -->
 	<dimen name="fineprint_size">12sp</dimen>
+	<dimen name="swipe_handle_size">48dp</dimen>
 </resources>
diff --git a/src/main/res/values-w585dp/dimens.xml b/src/main/res/values-w585dp/dimens.xml
index df6525e6ee31273f3ce5c0cb86a3afb8c0964e5a..be1f923915e8c835463d26595903fbd74760ae44 100644
--- a/src/main/res/values-w585dp/dimens.xml
+++ b/src/main/res/values-w585dp/dimens.xml
@@ -1,4 +1,4 @@
 <resources>
-	<dimen name="conversations_overview_width">416dp</dimen>
     <dimen name="activity_horizontal_margin">32dp</dimen>
+    <dimen name="swipe_handle_size">72dp</dimen>
 </resources>
diff --git a/src/main/res/values/dimens.xml b/src/main/res/values/dimens.xml
index bd749a0178e26021a8fa438fee47781cd3183f8e..1c067396b1e9a142d03896fe42a94d7d1433a87a 100644
--- a/src/main/res/values/dimens.xml
+++ b/src/main/res/values/dimens.xml
@@ -3,8 +3,8 @@
 	<dimen name="activity_horizontal_margin">8dp</dimen>
 	<dimen name="activity_vertical_margin">8dp</dimen>
 	<dimen name="infocard_padding">16dp</dimen>
-	<dimen name="conversations_overview_width">288dp</dimen>
 	<dimen name="image_button_padding">8dp</dimen>
 	<dimen name="fineprint_size">11sp</dimen>
 	<dimen name="menu_item_icon_size">24dp</dimen>
+	<dimen name="swipe_handle_size">24dp</dimen>
 </resources>