From 7e421825670b090d2fe035575769af751e908015 Mon Sep 17 00:00:00 2001
From: yueg <yueg@google.com>
Date: Fri, 19 Jan 2018 11:39:03 -0800
Subject: Bubble v2 layout changes.

To display expanded view properly with display size and font size set to
largest, we made the following changes:
- increase expanded view width
- use fixed expanded view button height
- remove blue background behind avatar

Test: manual
PiperOrigin-RevId: 182565152
Change-Id: Ie4cea2c42b66320767e283144ddd5d7c5ff40e1c
---
 .../newbubble/res/layout/new_bubble_base.xml       | 46 ++++++++++------------
 java/com/android/newbubble/res/values/styles.xml   |  7 ++--
 java/com/android/newbubble/res/values/values.xml   |  2 +-
 3 files changed, 25 insertions(+), 30 deletions(-)

(limited to 'java/com/android/newbubble/res')

diff --git a/java/com/android/newbubble/res/layout/new_bubble_base.xml b/java/com/android/newbubble/res/layout/new_bubble_base.xml
index 2b53b360e..c86d3bbcc 100644
--- a/java/com/android/newbubble/res/layout/new_bubble_base.xml
+++ b/java/com/android/newbubble/res/layout/new_bubble_base.xml
@@ -31,7 +31,7 @@
       android:animateLayoutChanges="true"
       android:clipChildren="false"
       android:clipToPadding="false">
-    <ViewAnimator
+    <FrameLayout
         android:id="@+id/bubble_button_primary"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
@@ -42,31 +42,25 @@
         android:contentDescription="@string/a11y_bubble_description"
         android:background="@drawable/bubble_shape_circle"
         android:measureAllChildren="false"
-        android:elevation="@dimen/bubble_elevation"
-        tools:backgroundTint="#FF0000AA">
-      <FrameLayout
-          android:id="@+id/bubble_icon_container"
-          android:layout_width="wrap_content"
-          android:layout_height="wrap_content">
-        <ImageView
-            android:id="@+id/bubble_icon_avatar"
-            android:layout_width="@dimen/bubble_size"
-            android:layout_height="@dimen/bubble_size"
-            tools:src="@android:drawable/ic_btn_speak_now"/>
-        <ImageView
-            android:id="@+id/bubble_icon_primary"
-            android:layout_width="@dimen/bubble_small_icon_size"
-            android:layout_height="@dimen/bubble_small_icon_size"
-            android:layout_gravity="bottom|right"
-            android:padding="@dimen/bubble_small_icon_padding"
-            android:tint="@android:color/white"
-            android:tintMode="src_in"
-            android:background="@drawable/bubble_shape_circle_small"
-            android:measureAllChildren="false"
-            tools:backgroundTint="#FF0000AA"
-            tools:src="@android:drawable/ic_btn_speak_now"/>
-      </FrameLayout>
-    </ViewAnimator>
+        android:elevation="@dimen/bubble_elevation">
+      <ImageView
+          android:id="@+id/bubble_icon_avatar"
+          android:layout_width="@dimen/bubble_size"
+          android:layout_height="@dimen/bubble_size"
+          tools:src="@android:drawable/ic_btn_speak_now"/>
+      <ImageView
+          android:id="@+id/bubble_icon_primary"
+          android:layout_width="@dimen/bubble_small_icon_size"
+          android:layout_height="@dimen/bubble_small_icon_size"
+          android:layout_gravity="bottom|right"
+          android:padding="@dimen/bubble_small_icon_padding"
+          android:tint="@android:color/white"
+          android:tintMode="src_in"
+          android:background="@drawable/bubble_shape_circle_small"
+          android:measureAllChildren="false"
+          tools:backgroundTint="#FF0000AA"
+          tools:src="@android:drawable/ic_btn_speak_now"/>
+    </FrameLayout>
   </RelativeLayout>
   <!-- The RelativeLayout below serves as boundary for @id/bubble_expanded_layout during animation -->
   <RelativeLayout
diff --git a/java/com/android/newbubble/res/values/styles.xml b/java/com/android/newbubble/res/values/styles.xml
index 6c138d645..5a900be6b 100644
--- a/java/com/android/newbubble/res/values/styles.xml
+++ b/java/com/android/newbubble/res/values/styles.xml
@@ -18,10 +18,11 @@
 <resources>
   <style name="CheckableButton">
     <item name="android:layout_width">@dimen/bubble_expanded_width</item>
-    <item name="android:layout_height">wrap_content</item>
+    <item name="android:layout_height">44dp</item>
 
-    <item name="android:paddingTop">@dimen/bubble_button_padding_vertical</item>
-    <item name="android:paddingBottom">@dimen/bubble_button_padding_vertical</item>
+    <item name="android:gravity">center_vertical</item>
+    <item name="android:maxLines">2</item>
+    <item name="android:ellipsize">end</item>
     <item name="android:paddingStart">@dimen/bubble_button_padding_horizontal</item>
     <item name="android:paddingEnd">@dimen/bubble_button_padding_horizontal</item>
     <item name="android:drawablePadding">@dimen/bubble_icon_padding</item>
diff --git a/java/com/android/newbubble/res/values/values.xml b/java/com/android/newbubble/res/values/values.xml
index 540f6653e..2e72c5e5e 100644
--- a/java/com/android/newbubble/res/values/values.xml
+++ b/java/com/android/newbubble/res/values/values.xml
@@ -36,7 +36,7 @@
 
   <dimen name="bubble_elevation">6dp</dimen>
   <dimen name="bubble_expanded_elevation">8dp</dimen>
-  <dimen name="bubble_expanded_width">160dp</dimen>
+  <dimen name="bubble_expanded_width">176dp</dimen>
   <dimen name="bubble_radius">16dp</dimen>
   <dimen name="bubble_expanded_separator_height">8dp</dimen>
   <dimen name="bubble_small_icon_size">24dp</dimen>
-- 
cgit v1.2.3