From e38918cd1f163b7fc4f1d824cce29b1097a9679c Mon Sep 17 00:00:00 2001
From: Dmitry Yunitsky <yunik@mapswithme.com>
Date: Tue, 3 Mar 2015 16:04:30 +0300
Subject: [PATCH] Layout corrections.

---
 .../res/layout-sw600dp-land/details_place_page.xml  | 11 +++++++----
 android/res/layout-sw600dp/details_place_page.xml   | 13 ++++++++-----
 android/res/layout-sw720dp/details_place_page.xml   | 11 +++++++----
 android/res/layout/details_place_page.xml           |  9 ++++++---
 4 files changed, 28 insertions(+), 16 deletions(-)

diff --git a/android/res/layout-sw600dp-land/details_place_page.xml b/android/res/layout-sw600dp-land/details_place_page.xml
index 14d7628781..719fe5656a 100644
--- a/android/res/layout-sw600dp-land/details_place_page.xml
+++ b/android/res/layout-sw600dp-land/details_place_page.xml
@@ -226,7 +226,6 @@
           android:layout_width="match_parent"
           android:layout_height="wrap_content"
           android:layout_below="@id/tv__bookmark_group_title"
-          android:layout_marginTop="@dimen/margin_tiny"
           android:layout_toRightOf="@id/iv__bookmark"
           android:gravity="center_vertical"
           android:orientation="horizontal">
@@ -236,8 +235,11 @@
             android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:layout_weight="1"
+            android:background="?android:selectableItemBackground"
             android:drawableRight="@drawable/ic_arrow_down"
             android:lineSpacingMultiplier="1.43"
+            android:paddingTop="@dimen/margin_small"
+            android:paddingBottom="@dimen/margin_small"
             android:paddingRight="@dimen/margin_small"
             android:textColor="@color/text_place_page"
             android:textSize="@dimen/place_page_subtitle"/>
@@ -246,7 +248,9 @@
             android:id="@+id/iv__bookmark_color"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginLeft="@dimen/margin_medium"
+            android:layout_marginLeft="@dimen/margin_small"
+            android:padding="@dimen/margin_small"
+            android:background="?android:selectableItemBackground"
             android:src="@drawable/color_picker_red_on"/>
       </LinearLayout>
 
@@ -254,7 +258,6 @@
           android:layout_width="match_parent"
           android:layout_height="1px"
           android:layout_below="@id/ll__group"
-          android:layout_marginTop="@dimen/margin_small"
           android:background="?android:attr/listDivider"/>
 
       <TextView
@@ -264,8 +267,8 @@
           android:layout_below="@id/ll__group"
           android:layout_marginTop="@dimen/margin_medium"
           android:layout_toRightOf="@id/iv__bookmark"
-          android:text="@string/description"
           android:lineSpacingMultiplier="1.67"
+          android:text="@string/description"
           android:textColor="@color/text_place_page_subtitle"
           android:textSize="@dimen/place_page_sub_subtitle"/>
 
diff --git a/android/res/layout-sw600dp/details_place_page.xml b/android/res/layout-sw600dp/details_place_page.xml
index 66406534bb..3499aad15a 100644
--- a/android/res/layout-sw600dp/details_place_page.xml
+++ b/android/res/layout-sw600dp/details_place_page.xml
@@ -193,8 +193,8 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_below="@id/ll__details_left"
-        android:visibility="invisible"
-        android:paddingTop="@dimen/margin_medium_and_half">
+        android:paddingTop="@dimen/margin_medium_and_half"
+        android:visibility="invisible">
 
       <View
           android:layout_width="match_parent"
@@ -259,7 +259,6 @@
           android:layout_width="match_parent"
           android:layout_height="wrap_content"
           android:layout_below="@id/tv__bookmark_group_title"
-          android:layout_marginTop="@dimen/margin_tiny"
           android:layout_toRightOf="@id/iv__bookmark"
           android:gravity="center_vertical"
           android:orientation="horizontal">
@@ -269,8 +268,11 @@
             android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:layout_weight="1"
+            android:background="?android:selectableItemBackground"
             android:drawableRight="@drawable/ic_arrow_down"
             android:lineSpacingMultiplier="1.43"
+            android:paddingTop="@dimen/margin_small"
+            android:paddingBottom="@dimen/margin_small"
             android:paddingRight="@dimen/margin_small"
             android:textColor="@color/text_place_page"
             android:textSize="@dimen/place_page_subtitle"/>
@@ -279,7 +281,9 @@
             android:id="@+id/iv__bookmark_color"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginLeft="@dimen/margin_medium"
+            android:layout_marginLeft="@dimen/margin_small"
+            android:padding="@dimen/margin_small"
+            android:background="?android:selectableItemBackground"
             android:src="@drawable/color_picker_red_on"/>
       </LinearLayout>
 
@@ -287,7 +291,6 @@
           android:layout_width="match_parent"
           android:layout_height="1px"
           android:layout_below="@id/ll__group"
-          android:layout_marginTop="@dimen/margin_small"
           android:background="?android:attr/listDivider"/>
 
       <TextView
diff --git a/android/res/layout-sw720dp/details_place_page.xml b/android/res/layout-sw720dp/details_place_page.xml
index dad9d6fd76..f54b3febc3 100644
--- a/android/res/layout-sw720dp/details_place_page.xml
+++ b/android/res/layout-sw720dp/details_place_page.xml
@@ -226,7 +226,6 @@
           android:layout_width="match_parent"
           android:layout_height="wrap_content"
           android:layout_below="@id/tv__bookmark_group_title"
-          android:layout_marginTop="@dimen/margin_tiny"
           android:layout_toRightOf="@id/iv__bookmark"
           android:gravity="center_vertical"
           android:orientation="horizontal">
@@ -235,18 +234,23 @@
             android:id="@+id/tv__bookmark_group"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
+            android:paddingTop="@dimen/margin_small"
+            android:paddingBottom="@dimen/margin_small"
+            android:paddingRight="@dimen/margin_small"
             android:layout_weight="1"
             android:drawableRight="@drawable/ic_arrow_down"
             android:lineSpacingMultiplier="1.43"
-            android:paddingRight="@dimen/margin_small"
+            android:background="?android:selectableItemBackground"
             android:textColor="@color/text_place_page"
             android:textSize="@dimen/place_page_subtitle"/>
 
         <ImageView
             android:id="@+id/iv__bookmark_color"
+            android:background="?android:selectableItemBackground"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginLeft="@dimen/margin_medium"
+            android:padding="@dimen/margin_small"
+            android:layout_marginLeft="@dimen/margin_small"
             android:src="@drawable/color_picker_red_on"/>
       </LinearLayout>
 
@@ -254,7 +258,6 @@
           android:layout_width="match_parent"
           android:layout_height="1px"
           android:layout_below="@id/ll__group"
-          android:layout_marginTop="@dimen/margin_small"
           android:background="?android:attr/listDivider"/>
 
       <TextView
diff --git a/android/res/layout/details_place_page.xml b/android/res/layout/details_place_page.xml
index bbee16e42c..2edff6b19f 100644
--- a/android/res/layout/details_place_page.xml
+++ b/android/res/layout/details_place_page.xml
@@ -234,7 +234,6 @@
           android:layout_width="match_parent"
           android:layout_height="wrap_content"
           android:layout_below="@id/tv__bookmark_group_title"
-          android:layout_marginTop="@dimen/margin_tiny"
           android:layout_toRightOf="@id/iv__bookmark"
           android:gravity="center_vertical"
           android:orientation="horizontal">
@@ -244,8 +243,11 @@
             android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:layout_weight="1"
+            android:background="?android:selectableItemBackground"
             android:drawableRight="@drawable/ic_arrow_down"
             android:lineSpacingMultiplier="1.43"
+            android:paddingTop="@dimen/margin_small"
+            android:paddingBottom="@dimen/margin_small"
             android:paddingRight="@dimen/margin_small"
             android:textColor="@color/text_place_page"
             android:textSize="@dimen/place_page_subtitle"/>
@@ -254,7 +256,9 @@
             android:id="@+id/iv__bookmark_color"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginLeft="@dimen/margin_medium"
+            android:layout_marginLeft="@dimen/margin_small"
+            android:padding="@dimen/margin_small"
+            android:background="?android:selectableItemBackground"
             android:src="@drawable/color_picker_red_on"/>
       </LinearLayout>
 
@@ -262,7 +266,6 @@
           android:layout_width="match_parent"
           android:layout_height="1px"
           android:layout_below="@id/ll__group"
-          android:layout_marginTop="@dimen/margin_small"
           android:background="?android:attr/listDivider"/>
 
       <TextView