From b82ab6fa8cb14d214b7e3476d5ba6aa230606af5 Mon Sep 17 00:00:00 2001
From: Doris Ling <dling@google.com>
Date: Thu, 23 Feb 2017 12:06:06 -0800
Subject: [PATCH] Change app detail header title to multi-line.

- allow multi line in app detail header title so that title text will
not be cut off
- fix layout so that the app links buttons and the title text won't
overlap

Change-Id: I9d35a20bc6630f89182023ba097781870184d122
Fix: 35675169
Test: Visual
---
 res/layout/app_details.xml | 61 +++++++++++++++++++-------------------
 1 file changed, 31 insertions(+), 30 deletions(-)

diff --git a/res/layout/app_details.xml b/res/layout/app_details.xml
index 263ce054334..6b6a5787c41 100644
--- a/res/layout/app_details.xml
+++ b/res/layout/app_details.xml
@@ -42,36 +42,6 @@
             android:layout_gravity="center_horizontal"
             android:antialias="true"/>
 
-        <LinearLayout
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:layout_toEndOf="@id/app_detail_icon"
-            android:paddingStart="24dp"
-            android:paddingEnd="24dp"
-            android:orientation="vertical">
-
-            <TextView
-                android:id="@+id/app_detail_title"
-                style="@style/TextAppearance.EntityHeaderTitle"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:singleLine="true"
-                android:ellipsize="marquee"
-                android:gravity="start"
-                android:paddingTop="8dp"/>
-
-            <TextView
-                android:id="@+id/app_detail_summary"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:gravity="start"
-                android:singleLine="true"
-                android:ellipsize="marquee"
-                android:textAppearance="@android:style/TextAppearance.Material.Body1"
-                android:textColor="?android:attr/textColorSecondary"/>
-
-        </LinearLayout>
-
         <LinearLayout
             android:id="@+id/app_detail_links"
             android:layout_width="wrap_content"
@@ -98,6 +68,37 @@
 
         </LinearLayout>
 
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_toEndOf="@id/app_detail_icon"
+            android:layout_toStartOf="@id/app_detail_links"
+            android:paddingStart="24dp"
+            android:paddingEnd="24dp"
+            android:orientation="vertical">
+
+            <TextView
+                android:id="@+id/app_detail_title"
+                style="@style/TextAppearance.EntityHeaderTitle"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:singleLine="false"
+                android:ellipsize="marquee"
+                android:gravity="start"
+                android:paddingTop="8dp"/>
+
+            <TextView
+                android:id="@+id/app_detail_summary"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:gravity="start"
+                android:singleLine="true"
+                android:ellipsize="marquee"
+                android:textAppearance="@android:style/TextAppearance.Material.Body1"
+                android:textColor="?android:attr/textColorSecondary"/>
+
+        </LinearLayout>
+
     </RelativeLayout>
 
     <TextView