From 98538c6efeae50838b8c84a90292d4a470db551a Mon Sep 17 00:00:00 2001
From: Alexander Sytnyk <asytnyk0@gmail.com>
Date: Fri, 6 Oct 2017 16:39:36 +0300
Subject: [PATCH] Fix landscape UI

---
 .../res/layout-land/fragment_plan_route.xml   | 118 ++++++++----------
 .../layout/fragment_map_markers_dialog.xml    |  78 ++++++------
 .../plus/mapmarkers/PlanRouteFragment.java    |   5 +-
 3 files changed, 91 insertions(+), 110 deletions(-)

diff --git a/OsmAnd/res/layout-land/fragment_plan_route.xml b/OsmAnd/res/layout-land/fragment_plan_route.xml
index 02306fc2f1..a969d9738c 100644
--- a/OsmAnd/res/layout-land/fragment_plan_route.xml
+++ b/OsmAnd/res/layout-land/fragment_plan_route.xml
@@ -16,95 +16,85 @@
         android:background="?attr/left_menu_view_bg"
         android:orientation="vertical">
 
-        <android.support.design.widget.AppBarLayout
+        <android.support.v7.widget.Toolbar
+            android:id="@+id/plan_route_toolbar"
             android:layout_width="match_parent"
             android:layout_height="@dimen/dashboard_map_toolbar"
-            tools:background="@color/actionbar_dark_color">
+            android:background="?attr/bg_color"
+            app:contentInsetLeft="54dp"
+            app:contentInsetStart="54dp">
 
-            <android.support.v7.widget.Toolbar
-                android:id="@+id/plan_route_toolbar"
+            <LinearLayout
                 android:layout_width="match_parent"
                 android:layout_height="match_parent"
-                app:contentInsetLeft="54dp"
-                app:contentInsetStart="54dp">
+                android:gravity="center_vertical"
+                android:orientation="horizontal">
 
                 <LinearLayout
-                    android:layout_width="match_parent"
-                    android:layout_height="match_parent"
-                    android:gravity="center_vertical"
-                    android:orientation="horizontal">
+                    android:layout_width="0dp"
+                    android:layout_height="wrap_content"
+                    android:layout_gravity="center_vertical"
+                    android:layout_weight="1"
+                    android:orientation="vertical">
 
                     <LinearLayout
-                        android:layout_width="0dp"
-                        android:layout_height="wrap_content"
-                        android:layout_gravity="center_vertical"
-                        android:layout_weight="1"
-                        android:orientation="vertical">
-
-                        <LinearLayout
-                            android:layout_width="match_parent"
-                            android:layout_height="match_parent">
-
-                            <TextView
-                                android:id="@+id/markers_distance_text_view"
-                                android:layout_width="wrap_content"
-                                android:layout_height="wrap_content"
-                                android:layout_gravity="center_vertical"
-                                android:layout_marginEnd="@dimen/measurement_tool_text_margin_small"
-                                android:layout_marginLeft="@dimen/measurement_tool_text_margin"
-                                android:layout_marginRight="@dimen/measurement_tool_text_margin_small"
-                                android:layout_marginStart="@dimen/measurement_tool_text_margin"
-                                android:ellipsize="end"
-                                android:maxLines="1"
-                                android:textAppearance="@style/TextAppearance.ListItemTitle"
-                                tools:text="1.39 km,"/>
-
-                            <TextView
-                                android:id="@+id/markers_time_text_view"
-                                android:layout_width="wrap_content"
-                                android:layout_height="wrap_content"
-                                android:layout_gravity="center_vertical"
-                                android:ellipsize="end"
-                                android:maxLines="1"
-                                android:textColor="?android:textColorSecondary"
-                                android:textSize="@dimen/default_list_text_size"
-                                tools:text="~ 45 min."/>
-                        </LinearLayout>
+                        android:layout_width="match_parent"
+                        android:layout_height="match_parent">
 
                         <TextView
-                            android:id="@+id/markers_count_text_view"
+                            android:id="@+id/markers_distance_text_view"
+                            android:layout_width="wrap_content"
+                            android:layout_height="wrap_content"
+                            android:layout_gravity="center_vertical"
+                            android:layout_marginEnd="@dimen/measurement_tool_text_margin_small"
+                            android:layout_marginRight="@dimen/measurement_tool_text_margin_small"
+                            android:ellipsize="end"
+                            android:maxLines="1"
+                            android:textAppearance="@style/TextAppearance.ListItemTitle"
+                            tools:text="1.39 km,"/>
+
+                        <TextView
+                            android:id="@+id/markers_time_text_view"
                             android:layout_width="wrap_content"
                             android:layout_height="wrap_content"
                             android:layout_gravity="center_vertical"
-                            android:layout_marginLeft="@dimen/measurement_tool_text_margin"
-                            android:layout_marginStart="@dimen/measurement_tool_text_margin"
                             android:ellipsize="end"
                             android:maxLines="1"
                             android:textColor="?android:textColorSecondary"
-                            android:textSize="@dimen/default_sub_text_size"
-                            tools:text="3 markers"/>
+                            android:textSize="@dimen/default_list_text_size"
+                            tools:text="~ 45 min."/>
                     </LinearLayout>
 
-                    <net.osmand.plus.widgets.TextViewEx
-                        android:id="@+id/options_button"
+                    <TextView
+                        android:id="@+id/markers_count_text_view"
                         android:layout_width="wrap_content"
-                        android:layout_height="match_parent"
-                        android:background="?attr/selectableItemBackground"
+                        android:layout_height="wrap_content"
+                        android:layout_gravity="center_vertical"
                         android:ellipsize="end"
-                        android:gravity="center_vertical"
                         android:maxLines="1"
-                        android:paddingLeft="16dp"
-                        android:paddingRight="16dp"
-                        android:text="@string/shared_string_options"
-                        android:textAllCaps="true"
-                        android:textColor="@color/color_white"
-                        android:textSize="@dimen/default_list_text_size"
-                        osmand:typeface="@string/font_roboto_regular"/>
+                        android:textColor="?android:textColorSecondary"
+                        android:textSize="@dimen/default_sub_text_size"
+                        tools:text="3 markers"/>
                 </LinearLayout>
 
-            </android.support.v7.widget.Toolbar>
+                <net.osmand.plus.widgets.TextViewEx
+                    android:id="@+id/options_button"
+                    android:layout_width="wrap_content"
+                    android:layout_height="match_parent"
+                    android:background="?attr/selectableItemBackground"
+                    android:ellipsize="end"
+                    android:gravity="center_vertical"
+                    android:maxLines="1"
+                    android:paddingLeft="16dp"
+                    android:paddingRight="16dp"
+                    android:text="@string/shared_string_options"
+                    android:textAllCaps="true"
+                    android:textColor="?attr/color_dialog_buttons"
+                    android:textSize="@dimen/default_list_text_size"
+                    osmand:typeface="@string/font_roboto_medium"/>
+            </LinearLayout>
 
-        </android.support.design.widget.AppBarLayout>
+        </android.support.v7.widget.Toolbar>
 
         <ProgressBar
             android:id="@+id/snap_to_road_progress_bar"
diff --git a/OsmAnd/res/layout/fragment_map_markers_dialog.xml b/OsmAnd/res/layout/fragment_map_markers_dialog.xml
index f513542ed8..efdbeb931b 100644
--- a/OsmAnd/res/layout/fragment_map_markers_dialog.xml
+++ b/OsmAnd/res/layout/fragment_map_markers_dialog.xml
@@ -2,60 +2,54 @@
 <LinearLayout
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:osmand="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    xmlns:osmand="http://schemas.android.com/tools"
     android:orientation="vertical">
 
-    <android.support.design.widget.AppBarLayout
+    <android.support.v7.widget.Toolbar
+        android:id="@+id/map_markers_toolbar"
         android:layout_width="match_parent"
-        android:layout_height="@dimen/dashboard_map_toolbar">
+        android:layout_height="@dimen/dashboard_map_toolbar"
+        app:contentInsetLeft="54dp"
+        app:contentInsetStart="54dp">
 
-        <android.support.v7.widget.Toolbar
-            android:id="@+id/map_markers_toolbar"
+        <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            app:contentInsetLeft="54dp"
-            app:contentInsetStart="54dp">
+            android:gravity="center_vertical"
+            android:orientation="horizontal">
 
-            <LinearLayout
-                android:orientation="horizontal"
-                android:layout_width="match_parent"
+            <net.osmand.plus.widgets.TextViewEx
+                android:layout_width="0dp"
                 android:layout_height="match_parent"
-                android:gravity="center_vertical">
+                android:layout_weight="1"
+                android:ellipsize="end"
+                android:gravity="center_vertical"
+                android:maxLines="1"
+                android:text="@string/map_markers"
+                android:textColor="@color/color_white"
+                android:textSize="@dimen/dialog_header_text_size"
+                osmand:typeface="@string/font_roboto_medium"/>
 
-                <net.osmand.plus.widgets.TextViewEx
-                    android:layout_width="0dp"
-                    android:layout_weight="1"
-                    android:layout_height="match_parent"
-                    android:ellipsize="end"
-                    android:maxLines="1"
-                    android:gravity="center_vertical"
-                    android:text="@string/map_markers"
-                    android:textColor="@color/color_white"
-                    osmand:typeface="@string/font_roboto_medium"
-                    android:textSize="@dimen/dialog_header_text_size"/>
+            <net.osmand.plus.widgets.TextViewEx
+                android:id="@+id/options_button"
+                android:layout_width="wrap_content"
+                android:layout_height="match_parent"
+                android:background="?attr/selectableItemBackground"
+                android:ellipsize="end"
+                android:gravity="center_vertical"
+                android:maxLines="1"
+                android:paddingLeft="16dp"
+                android:paddingRight="16dp"
+                android:text="@string/shared_string_options"
+                android:textAllCaps="true"
+                android:textColor="@color/color_white"
+                android:textSize="@dimen/default_list_text_size"
+                osmand:typeface="@string/font_roboto_regular"/>
+        </LinearLayout>
 
-                <net.osmand.plus.widgets.TextViewEx
-                    android:id="@+id/options_button"
-                    android:layout_width="wrap_content"
-                    android:layout_height="match_parent"
-                    android:gravity="center_vertical"
-                    android:ellipsize="end"
-                    android:maxLines="1"
-                    android:paddingRight="16dp"
-                    android:paddingLeft="16dp"
-                    android:text="@string/shared_string_options"
-                    osmand:typeface="@string/font_roboto_regular"
-                    android:textAllCaps="true"
-                    android:textSize="@dimen/default_list_text_size"
-                    android:textColor="@color/color_white"
-                    android:background="?attr/selectableItemBackground"/>
-            </LinearLayout>
-
-        </android.support.v7.widget.Toolbar>
-
-    </android.support.design.widget.AppBarLayout>
+    </android.support.v7.widget.Toolbar>
 
     <FrameLayout
         android:layout_width="match_parent"
diff --git a/OsmAnd/src/net/osmand/plus/mapmarkers/PlanRouteFragment.java b/OsmAnd/src/net/osmand/plus/mapmarkers/PlanRouteFragment.java
index 8ebe72239f..cdb1f4d5e2 100644
--- a/OsmAnd/src/net/osmand/plus/mapmarkers/PlanRouteFragment.java
+++ b/OsmAnd/src/net/osmand/plus/mapmarkers/PlanRouteFragment.java
@@ -233,10 +233,7 @@ public class PlanRouteFragment extends Fragment {
 			});
 		} else {
 			Toolbar toolbar = (Toolbar) mainView.findViewById(R.id.plan_route_toolbar);
-			if (nightMode) {
-				toolbar.setBackgroundColor(ContextCompat.getColor(getContext(), R.color.actionbar_dark_color));
-			}
-			toolbar.setNavigationIcon(iconsCache.getIcon(R.drawable.ic_arrow_back));
+			toolbar.setNavigationIcon(getContentIcon(R.drawable.ic_arrow_back));
 			toolbar.setNavigationOnClickListener(new View.OnClickListener() {
 				@Override
 				public void onClick(View view) {