From 0362dc6c1df7e33e9c25abb5b8aed293c67f1f84 Mon Sep 17 00:00:00 2001
From: Ishan Khanna <ishan1604@gmail.com>
Date: Wed, 4 Mar 2015 15:47:44 +0530
Subject: [PATCH] Orange Toolbar Added Fixes #1088

---
 .../ui/CertifyFingerprintActivity.java        | 21 -------------
 .../keychain/ui/CertifyKeyActivity.java       | 21 -------------
 .../layout/certify_fingerprint_activity.xml   |  2 +-
 .../main/res/layout/certify_key_activity.xml  |  2 +-
 .../res/layout/toolbar_standalone_orange.xml  | 31 +++++++++++++++++++
 5 files changed, 33 insertions(+), 44 deletions(-)
 create mode 100644 OpenKeychain/src/main/res/layout/toolbar_standalone_orange.xml

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyFingerprintActivity.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyFingerprintActivity.java
index fcfcc9cb2..b7c80c1ed 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyFingerprintActivity.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyFingerprintActivity.java
@@ -19,10 +19,7 @@ package org.sufficientlysecure.keychain.ui;
 
 import android.net.Uri;
 import android.os.Bundle;
-import android.support.v7.widget.Toolbar;
 import android.view.View;
-import android.widget.ImageView;
-import android.widget.RelativeLayout;
 
 import org.sufficientlysecure.keychain.Constants;
 import org.sufficientlysecure.keychain.R;
@@ -58,7 +55,6 @@ public class CertifyFingerprintActivity extends BaseActivity {
     @Override
     protected void initLayout() {
         setContentView(R.layout.certify_fingerprint_activity);
-        changeToolbarColor();
     }
 
     private void startFragment(Bundle savedInstanceState, Uri dataUri) {
@@ -81,21 +77,4 @@ public class CertifyFingerprintActivity extends BaseActivity {
         getSupportFragmentManager().executePendingTransactions();
     }
 
-    /**
-     * Changes the color of our ToolBar.
-     *
-     * Currently Set to ORANGE
-     */
-    private void changeToolbarColor() {
-        RelativeLayout mToolBarInclude = (RelativeLayout) findViewById(R.id.toolbar_include);
-
-        // Changes the color of the Status Bar strip
-        ImageView mStatusBar = (ImageView) mToolBarInclude.findViewById(R.id.status_bar);
-        mStatusBar.setBackgroundResource(getResources().getColor(R.color.android_orange_dark));
-
-        // Changes the color of our Tool Bar
-        Toolbar toolbar = (Toolbar) mToolBarInclude.findViewById(R.id.toolbar);
-        toolbar.setBackgroundResource(getResources().getColor(R.color.android_orange_light));
-    }
-
 }
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyKeyActivity.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyKeyActivity.java
index a2e717ed2..1fb88b182 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyKeyActivity.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CertifyKeyActivity.java
@@ -18,10 +18,6 @@
 
 package org.sufficientlysecure.keychain.ui;
 
-import android.support.v7.widget.Toolbar;
-import android.widget.ImageView;
-import android.widget.RelativeLayout;
-
 import org.sufficientlysecure.keychain.R;
 
 /**
@@ -36,23 +32,6 @@ public class CertifyKeyActivity extends BaseActivity {
     @Override
     protected void initLayout() {
         setContentView(R.layout.certify_key_activity);
-        changeToolbarColor();
     }
 
-    /**
-     * Changes the color of our ToolBar.
-     *
-     * Currently Set to ORANGE
-     */
-    private void changeToolbarColor() {
-        RelativeLayout mToolBarInclude = (RelativeLayout) findViewById(R.id.toolbar_include);
-
-        // Changes the color of the Status Bar strip
-        ImageView mStatusBar = (ImageView) mToolBarInclude.findViewById(R.id.status_bar);
-        mStatusBar.setBackgroundResource(getResources().getColor(R.color.android_orange_dark));
-
-        // Changes the color of our Tool Bar
-        Toolbar toolbar = (Toolbar) mToolBarInclude.findViewById(R.id.toolbar);
-        toolbar.setBackgroundResource(getResources().getColor(R.color.android_orange_light));
-    }
 }
diff --git a/OpenKeychain/src/main/res/layout/certify_fingerprint_activity.xml b/OpenKeychain/src/main/res/layout/certify_fingerprint_activity.xml
index ec91d1455..764b2eac0 100644
--- a/OpenKeychain/src/main/res/layout/certify_fingerprint_activity.xml
+++ b/OpenKeychain/src/main/res/layout/certify_fingerprint_activity.xml
@@ -7,7 +7,7 @@
 
     <include
         android:id="@+id/toolbar_include"
-        layout="@layout/toolbar_standalone" />
+        layout="@layout/toolbar_standalone_orange" />
 
     <LinearLayout
         android:layout_below="@id/toolbar_include"
diff --git a/OpenKeychain/src/main/res/layout/certify_key_activity.xml b/OpenKeychain/src/main/res/layout/certify_key_activity.xml
index fab8bc2d2..56dca302f 100644
--- a/OpenKeychain/src/main/res/layout/certify_key_activity.xml
+++ b/OpenKeychain/src/main/res/layout/certify_key_activity.xml
@@ -5,7 +5,7 @@
 
     <include
         android:id="@+id/toolbar_include"
-        layout="@layout/toolbar_standalone" />
+        layout="@layout/toolbar_standalone_orange" />
 
     <LinearLayout
         android:layout_below="@id/toolbar_include"
diff --git a/OpenKeychain/src/main/res/layout/toolbar_standalone_orange.xml b/OpenKeychain/src/main/res/layout/toolbar_standalone_orange.xml
new file mode 100644
index 000000000..352da739b
--- /dev/null
+++ b/OpenKeychain/src/main/res/layout/toolbar_standalone_orange.xml
@@ -0,0 +1,31 @@
+<?xml version="1.0" encoding="utf-8"?>
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:id="@+id/toolbar_include"
+    android:elevation="4dp"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content">
+
+    <!--
+        We always have windowTranslucentStatus=true to get under the status bar.
+        Thus this ImageView is the part under the status bar!
+    -->
+    <ImageView
+        android:id="@+id/status_bar"
+        android:layout_width="match_parent"
+        android:layout_height="@dimen/statusbar_height"
+        android:background="@color/android_orange_dark" />
+
+    <android.support.v7.widget.Toolbar
+        android:id="@+id/toolbar"
+        android:layout_below="@+id/status_bar"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:minHeight="?attr/actionBarSize"
+        android:background="@color/android_orange_light"
+        app:theme="@style/ThemeOverlay.AppCompat.Dark.ActionBar"
+        app:popupTheme="@style/ThemeOverlay.AppCompat.Light"
+        tools:ignore="UnusedAttribute" />
+
+</RelativeLayout>