Bug 953288 - Limit length of URLs for display. r=lucasr, a=bajaj
authorJim Chen <nchen@mozilla.com>
Tue, 07 Jan 2014 12:21:54 -0600
changeset 167871 bbb29df330c58251dac48e3f0425324ae99e7401
parent 167870 3d79e4c4b3542a53a9c23297b84eb6be7c08787c
child 167872 dad9fe3bf6ee48c9d1f81c280ca7669bcb2e9d4c
push id428
push userbbajaj@mozilla.com
push dateTue, 28 Jan 2014 00:16:25 +0000
treeherdermozilla-release@cd72a7ff3a75 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerslucasr, bajaj
bugs953288
milestone27.0
Bug 953288 - Limit length of URLs for display. r=lucasr, a=bajaj
mobile/android/base/resources/layout/remote_tabs_child.xml
mobile/android/base/resources/layout/two_line_page_row.xml
--- a/mobile/android/base/resources/layout/remote_tabs_child.xml
+++ b/mobile/android/base/resources/layout/remote_tabs_child.xml
@@ -17,11 +17,12 @@
               android:layout_height="wrap_content"
               style="@style/TabRowTextAppearance"
               android:textSize="18sp"/>
 
     <TextView android:id="@+id/url"
               android:layout_width="fill_parent"
               android:layout_height="wrap_content"
               style="@style/TabRowTextAppearance.Url"
-              android:textSize="14sp"/>
+              android:textSize="14sp"
+              android:maxLength="1024"/>
 
 </LinearLayout>
--- a/mobile/android/base/resources/layout/two_line_page_row.xml
+++ b/mobile/android/base/resources/layout/two_line_page_row.xml
@@ -23,13 +23,14 @@
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 gecko:fadeWidth="30dp"/>
 
         <TextView android:id="@+id/url"
                   style="@style/Widget.TwoLinePageRow.Url"
                   android:layout_width="fill_parent"
                   android:layout_height="wrap_content"
-                  android:drawablePadding="5dp"/>
+                  android:drawablePadding="5dp"
+                  android:maxLength="1024"/>
 
     </LinearLayout>
 
 </merge>