Bug 1110461 - Disable estimated reading time in reading list. r=mcomella, a=sledru
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Thu, 11 Dec 2014 13:44:49 -0800
changeset 242458 68f54ed7e2ac12446e7c03a7daaad61d347ebd0b
parent 242457 0a8761cd24cff24312155badc8f16b8c0f4ed7fc
child 242459 bdb2bea5ff32f2bd98f7a60b66127ded3013e133
push id4311
push userraliiev@mozilla.com
push dateMon, 12 Jan 2015 19:37:41 +0000
treeherdermozilla-beta@150c9fed433b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmcomella, sledru
bugs1110461
milestone36.0a2
Bug 1110461 - Disable estimated reading time in reading list. r=mcomella, a=sledru
mobile/android/base/home/ReadingListRow.java
mobile/android/base/resources/layout/reading_list_row_view.xml
--- a/mobile/android/base/home/ReadingListRow.java
+++ b/mobile/android/base/home/ReadingListRow.java
@@ -56,23 +56,25 @@ public class ReadingListRow extends Line
         }
 
         final int titleIndex = cursor.getColumnIndexOrThrow(ReadingListItems.TITLE);
         title.setText(cursor.getString(titleIndex));
 
         final int excerptIndex = cursor.getColumnIndexOrThrow(ReadingListItems.EXCERPT);
         excerpt.setText(cursor.getString(excerptIndex));
 
+        /* Disabled until UX issues are fixed (see bug 1110461).
         final int lengthIndex = cursor.getColumnIndexOrThrow(ReadingListItems.LENGTH);
         final int minutes = getEstimatedReadTime(cursor.getInt(lengthIndex));
         if (minutes <= 60) {
             readTime.setText(resources.getString(R.string.reading_list_time_minutes, minutes));
         } else {
             readTime.setText(resources.getString(R.string.reading_list_time_over_an_hour));
         }
+        */
     }
 
     /**
      * Calculates the estimated time to read an article based on its length.
      *
      * @param length of the article (in characters)
      * @return estimated time to read the article (in minutes)
      */
--- a/mobile/android/base/resources/layout/reading_list_row_view.xml
+++ b/mobile/android/base/resources/layout/reading_list_row_view.xml
@@ -29,11 +29,12 @@
 
     </LinearLayout>
 
     <TextView
         android:id="@+id/read_time"
         android:layout_width="64dp"
         android:layout_height="match_parent"
         android:gravity="center"
+        android:visibility="gone"
         style="@style/Widget.ReadingListRow.ReadTime" />
 
 </merge>