From c91419614f073226b06ae0cd13a74967d76a5a04 Mon Sep 17 00:00:00 2001
From: Jay Wang <jaywang@codeaurora.org>
Date: Tue, 13 Sep 2016 18:17:44 -0700
Subject: SnapdragonGallery: Update UI for TruePortrait effects

Make UI adjustments based on feedback from UX team
for TruePortrait effects.

CRs-Fixed: 1066553
Change-Id: If5cf78af60daf81a4abf76e8d85534322760ccc8
---
 res/drawable/tp_mask_toggle.xml                    | 34 ++++++++++++++++++++++
 res/layout/do_not_show_again_dialog.xml            | 12 ++++++--
 .../filtershow_actionbar_trueportrait_mask.xml     |  8 +++++
 res/values/filtershow_strings.xml                  |  2 +-
 4 files changed, 53 insertions(+), 3 deletions(-)
 create mode 100644 res/drawable/tp_mask_toggle.xml

(limited to 'res')

diff --git a/res/drawable/tp_mask_toggle.xml b/res/drawable/tp_mask_toggle.xml
new file mode 100644
index 000000000..29abd41fc
--- /dev/null
+++ b/res/drawable/tp_mask_toggle.xml
@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  Copyright (c) 2016, The Linux Foundation. All rights reserved.
+
+  Redistribution and use in source and binary forms, with or without
+  modification, are permitted provided that the following conditions are
+  met:
+      * Redistributions of source code must retain the above copyright
+        notice, this list of conditions and the following disclaimer.
+      * Redistributions in binary form must reproduce the above
+        copyright notice, this list of conditions and the following
+        disclaimer in the documentation and/or other materials provided
+        with the distribution.
+      * Neither the name of The Linux Foundation nor the names of its
+        contributors may be used to endorse or promote products derived
+        from this software without specific prior written permission.
+
+  THIS SOFTWARE IS PROVIDED "AS IS" AND ANY EXPRESS OR IMPLIED
+  WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+  MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT
+  ARE DISCLAIMED.  IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS
+  BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+  CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+  SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+  BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+  WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
+  OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
+  IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+-->
+
+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+    <item android:drawable="@android:color/holo_blue_light" android:state_checked="true"></item>
+    <item android:drawable="@color/cache_placeholder" android:state_checked="false"></item>
+</selector>
\ No newline at end of file
diff --git a/res/layout/do_not_show_again_dialog.xml b/res/layout/do_not_show_again_dialog.xml
index 8ddfdc191..d3d229740 100644
--- a/res/layout/do_not_show_again_dialog.xml
+++ b/res/layout/do_not_show_again_dialog.xml
@@ -30,19 +30,27 @@
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
+    android:paddingTop="8dip"
+    android:paddingBottom="8dip"
+    android:paddingEnd="?android:attr/dialogPreferredPadding"
+    android:paddingStart="?android:attr/dialogPreferredPadding"
     android:orientation="vertical" >
 
     <TextView
         android:id="@+id/message"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_margin="8dp" />
+        style="@android:style/TextAppearance.Material.Subhead" />
+
+    <Space
+        android:layout_width="0dp"
+        android:layout_height="5dp"/>
 
     <CheckBox
         android:id="@+id/do_not_show_chk"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_margin="8dp"
+        style="@android:style/TextAppearance.Material.Subhead"
         android:text="@string/do_not_show_again" />
 
 </LinearLayout>
\ No newline at end of file
diff --git a/res/layout/filtershow_actionbar_trueportrait_mask.xml b/res/layout/filtershow_actionbar_trueportrait_mask.xml
index b922dbd2d..f992e76a8 100644
--- a/res/layout/filtershow_actionbar_trueportrait_mask.xml
+++ b/res/layout/filtershow_actionbar_trueportrait_mask.xml
@@ -52,15 +52,23 @@
             android:id="@+id/maskForeground"
             android:layout_width="wrap_content"
             android:layout_height="match_parent"
+            android:background="@drawable/tp_mask_toggle"
+            android:textColor="@color/primary_text"
             android:paddingEnd="16dp"
             android:paddingStart="16dp"
             android:textOff="@string/foreground"
             android:textOn="@string/foreground" />
 
+        <Space
+	        android:layout_width="5dp"
+	        android:layout_height="0dp"/>
+
         <ToggleButton
             android:id="@+id/maskBackground"
             android:layout_width="wrap_content"
             android:layout_height="match_parent"
+            android:background="@drawable/tp_mask_toggle"
+            android:textColor="@color/primary_text"
             android:paddingEnd="16dp"
             android:paddingStart="16dp"
             android:textOff="@string/background"
diff --git a/res/values/filtershow_strings.xml b/res/values/filtershow_strings.xml
index 81b7ba535..7c0212e7e 100644
--- a/res/values/filtershow_strings.xml
+++ b/res/values/filtershow_strings.xml
@@ -352,7 +352,7 @@
     <string name="fusion_pick_underlay">Pick Underlay</string>
     <string name="dualcam_no_segment_toast">No segment found at this point</string>
     <string name="dualcam_filter_not_supported">Dual camera filters not supported for this image</string>
-    <string name="trueportrait_intro">Allows you to enhance photos of people. You can apply effects to the background, such "blur" and "sketch". You can also merge two photos together to create interesting backgrounds for your portrait photos.</string>
+    <string name="trueportrait_intro">Allows you to enhance photos of people. You can apply effects to the background, such as \"blur\" and \"sketch\". You can also merge two photos together to create interesting backgrounds for your portrait photos.</string>
     <string name="trueportrait_no_face">True Portrait effects cannot be applied to this photo. Face not detected.</string>
     <string name="trueportrait_edit_intro">With the Touch-Up tool, you can edit the foreground and the background. You can user your finger to paint in any areas of the foreground or background that were missed.</string>
     <string name="trueportrait_edit_foreground_toast">Use your finger to paint in any areas of the foreground that were missed.</string>
-- 
cgit v1.2.3