From 03c97e912efc8a1f4c4df2e82e21ceccf6a4b66d Mon Sep 17 00:00:00 2001
From: Alex Ning <chineseperson5@gmail.com>
Date: Tue, 24 Sep 2019 08:51:56 +0800
Subject: Minor UI tweaks. Fix UI issues in dark theme. Fix settings activity
 title.

---
 app/src/main/res/layout/activity_search.xml        | 53 +++++++++-------------
 app/src/main/res/layout/activity_search_result.xml |  3 +-
 app/src/main/res/layout/app_bar_main.xml           |  3 +-
 3 files changed, 24 insertions(+), 35 deletions(-)

(limited to 'app/src/main/res/layout')

diff --git a/app/src/main/res/layout/activity_search.xml b/app/src/main/res/layout/activity_search.xml
index 15c3ba9e..d83a52ea 100644
--- a/app/src/main/res/layout/activity_search.xml
+++ b/app/src/main/res/layout/activity_search.xml
@@ -11,42 +11,33 @@
         android:layout_height="wrap_content"
         android:theme="@style/AppTheme.AppBarOverlay">
 
-        <com.google.android.material.appbar.CollapsingToolbarLayout
-            android:id="@+id/collapsing_toolbar_layout_search_activity"
+        <FrameLayout
+            android:id="@+id/toolbar_frame_layout_search_activity"
             android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            app:titleEnabled="false"
-            app:toolbarId="@+id/toolbar">
+            android:layout_height="wrap_content">
 
-            <FrameLayout
-                android:id="@+id/toolbar_frame_layout_search_activity"
+            <androidx.appcompat.widget.Toolbar
+                android:id="@+id/toolbar"
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content">
+                android:layout_height="?attr/actionBarSize"
+                app:navigationIcon="?attr/homeAsUpIndicator"
+                app:popupTheme="@style/AppTheme.PopupOverlay" />
 
-                <androidx.appcompat.widget.Toolbar
-                    android:id="@+id/toolbar"
-                    android:layout_width="match_parent"
-                    android:layout_height="?attr/actionBarSize"
-                    app:navigationIcon="?attr/homeAsUpIndicator"
-                    app:popupTheme="@style/AppTheme.PopupOverlay" />
-
-                <com.ferfalk.simplesearchview.SimpleSearchView
-                    android:id="@+id/search_view_search_activity"
-                    android:layout_width="match_parent"
-                    android:layout_height="match_parent"
-                    android:textColor="@android:color/white"
-                    app:backIconAlpha="1"
-                    app:backIconTint="@android:color/white"
-                    app:cursorColor="@android:color/white"
-                    app:hintColor="#E0E0E0"
-                    app:iconsAlpha="1"
-                    app:iconsTint="@android:color/white"
-                    app:searchBackground="@color/colorPrimary"
-                    app:voiceSearch="true" />
-
-            </FrameLayout>
+            <com.ferfalk.simplesearchview.SimpleSearchView
+                android:id="@+id/search_view_search_activity"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:textColor="@android:color/white"
+                app:backIconAlpha="1"
+                app:backIconTint="@android:color/white"
+                app:cursorColor="@android:color/white"
+                app:hintColor="#E0E0E0"
+                app:iconsAlpha="1"
+                app:iconsTint="@android:color/white"
+                app:searchBackground="@color/colorPrimary"
+                app:voiceSearch="true" />
 
-        </com.google.android.material.appbar.CollapsingToolbarLayout>
+        </FrameLayout>
 
     </com.google.android.material.appbar.AppBarLayout>
 
diff --git a/app/src/main/res/layout/activity_search_result.xml b/app/src/main/res/layout/activity_search_result.xml
index ca7c4e84..839820c6 100644
--- a/app/src/main/res/layout/activity_search_result.xml
+++ b/app/src/main/res/layout/activity_search_result.xml
@@ -26,7 +26,6 @@
                     android:id="@+id/toolbar_search_result_activity"
                     android:layout_width="match_parent"
                     android:layout_height="?attr/actionBarSize"
-                    android:background="?attr/colorPrimary"
                     app:popupTheme="@style/AppTheme.PopupOverlay" />
 
                 <com.ferfalk.simplesearchview.SimpleSearchView
@@ -51,7 +50,7 @@
             android:id="@+id/tab_layout_search_result_activity"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:background="@color/colorPrimary"
+            android:background="@color/greyTabBackgroundColor"
             app:layout_scrollFlags="scroll|enterAlways"
             app:tabGravity="fill"
             app:tabMode="fixed"
diff --git a/app/src/main/res/layout/app_bar_main.xml b/app/src/main/res/layout/app_bar_main.xml
index 5a0f4287..b834c1ff 100644
--- a/app/src/main/res/layout/app_bar_main.xml
+++ b/app/src/main/res/layout/app_bar_main.xml
@@ -25,7 +25,6 @@
                 android:id="@+id/toolbar"
                 android:layout_width="match_parent"
                 android:layout_height="?attr/actionBarSize"
-                android:background="?attr/colorPrimary"
                 app:layout_collapseMode="pin"
                 app:layout_scrollFlags="scroll|enterAlways"
                 app:popupTheme="@style/AppTheme.PopupOverlay" />
@@ -37,7 +36,7 @@
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_gravity="bottom"
-            android:background="@color/colorPrimary"
+            android:background="@color/greyTabBackgroundColor"
             app:layout_scrollFlags="scroll|enterAlways"
             app:tabGravity="fill"
             app:tabIndicatorColor="@android:color/white"
-- 
cgit v1.2.3