From 3776553208fc80690e281f325d93d7bc6a779be4 Mon Sep 17 00:00:00 2001
From: Sagar Dhawan <sdhawan@codeaurora.org>
Date: Fri, 31 Jul 2015 18:46:35 -0700
Subject: Fix crash when viewing pages offline on Tablets

Fixed multiple issues with Snapshot tabs on
tablets.
---
 res/layout-sw600dp/snapshot_item.xml      | 42 +++++++++-------------
 res/layout-sw600dp/title_bar_snapshot.xml | 58 ++++++++++++++++++++-----------
 2 files changed, 53 insertions(+), 47 deletions(-)

(limited to 'res')

diff --git a/res/layout-sw600dp/snapshot_item.xml b/res/layout-sw600dp/snapshot_item.xml
index 41b465b8..c3ce866c 100644
--- a/res/layout-sw600dp/snapshot_item.xml
+++ b/res/layout-sw600dp/snapshot_item.xml
@@ -28,7 +28,7 @@
         android:cropToPadding="true"
         android:background="@drawable/border_thumb_bookmarks_widget_holo" />
     <TextView
-        android:id="@+id/title"
+        android:id="@+id/date"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_alignBottom="@id/thumb"
@@ -37,41 +37,31 @@
         android:background="@drawable/overlay_url_bookmark_widget_holo"
         android:singleLine="true"
         android:ellipsize="end"
-        android:textSize="12sp"
+        android:textSize="14sp"
         android:typeface="sans"
-        android:textColor="@android:color/white"
+        android:textColor="#AAAAAA"
         android:paddingStart="6dip"
         android:paddingEnd="2dip"
         android:gravity="center_vertical" />
-    <ImageView
-        android:id="@+id/divider"
-        android:src="?android:attr/dividerVertical"
+    <TextView android:id="@+id/title"
         android:layout_width="wrap_content"
-        android:layout_height="24dip"
+        android:layout_height="wrap_content"
         android:layout_below="@+id/thumb"
         android:layout_alignStart="@+id/thumb"
-        android:scaleType="fitXY"
-        android:layout_marginTop="6dip" />
-    <TextView android:id="@+id/date"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_toEndOf="@id/divider"
-        android:layout_alignTop="@id/divider"
-        android:layout_alignBottom="@id/divider"
+        android:layout_alignEnd="@+id/thumb"
+        android:layout_marginTop="6dip"
         android:paddingStart="8dip"
         android:gravity="center_vertical"
         android:typeface="sans"
-        android:textSize="14sp"
-        android:textColor="#AAAAAA" />
-    <TextView android:id="@+id/size"
+        android:textAppearance="@style/BookmarkText"
+        android:lines="2" />
+    <ImageView
+        android:id="@+id/divider"
+        android:src="?android:attr/dividerVertical"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_alignTop="@id/divider"
-        android:layout_alignBottom="@id/divider"
-        android:layout_alignEnd="@+id/thumb"
-        android:paddingEnd="2dip"
-        android:gravity="center_vertical"
-        android:typeface="sans"
-        android:textSize="14sp"
-        android:textColor="#AAAAAA" />
+        android:layout_alignStart="@+id/title"
+        android:layout_alignTop="@+id/title"
+        android:layout_alignBottom="@+id/title"
+        android:scaleType="fitXY" />
 </RelativeLayout>
diff --git a/res/layout-sw600dp/title_bar_snapshot.xml b/res/layout-sw600dp/title_bar_snapshot.xml
index 434b2f14..8c8189ab 100644
--- a/res/layout-sw600dp/title_bar_snapshot.xml
+++ b/res/layout-sw600dp/title_bar_snapshot.xml
@@ -18,37 +18,53 @@
     xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
+    android:background="@android:color/white"
     android:orientation="horizontal" >
     <ImageView
-        android:id="@+id/date_icon"
-        android:src="@drawable/ic_suggest_history_normal"
+        android:id="@+id/reader_icon"
+        android:src="@drawable/ic_deco_reader_mode_normal"
         android:layout_width="wrap_content"
         android:layout_height="match_parent"
+        android:visibility="gone"
+        android:paddingEnd="16dip"
         android:paddingStart="16dip" />
-    <TextView
-        android:id="@+id/date"
+    <ImageView
+        android:id="@+id/snapshot_icon"
+        android:src="@drawable/ic_suggest_history_normal"
         android:layout_width="wrap_content"
         android:layout_height="match_parent"
         android:paddingStart="8dip"
-        android:singleLine="true"
-        android:gravity="center_vertical"
-        android:textAppearance="?android:attr/textAppearanceMedium" />
-    <TextView
-        android:id="@+id/title"
-        android:layout_width="0dip"
+        android:paddingEnd="8dip"/>
+    <ImageView
+        android:id="@+id/favicon"
+        android:layout_width="52dip"
         android:layout_height="match_parent"
-        android:layout_weight="1"
         android:paddingStart="16dip"
-        android:singleLine="true"
-        android:gravity="center_vertical"
-        android:textAppearance="?android:attr/textAppearanceMedium" />
+        android:paddingEnd="16dip"
+        android:scaleType="fitCenter" />
+    <FrameLayout
+        android:id="@+id/toggle_container"
+        android:layout_width="0dip"
+        android:layout_height="match_parent"
+        android:layout_weight="1">
+        <TextView
+            android:id="@+id/title"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            android:singleLine="true"
+            android:gravity="center_vertical"
+            android:textAppearance="?android:attr/textAppearanceMedium" />
+        <TextView
+            android:id="@+id/date"
+            android:layout_width="wrap_content"
+            android:layout_height="match_parent"
+            android:singleLine="true"
+            android:gravity="center_vertical"
+            android:textAppearance="?android:attr/textAppearanceMedium" />
+    </FrameLayout>
 
     <ImageButton
-        android:id="@+id/all_btn"
-        style="@style/NavButton"
-        android:layout_width="wrap_content"
-        android:contentDescription="@string/accessibility_button_bookmarks"
-        android:paddingEnd="16dip"
-        android:scaleType="center"
-        android:src="@drawable/ic_deco_bookmarks_normal" />
+        android:id="@+id/more"
+        style="@style/OverflowButton" />
+
 </com.android.browser.SnapshotBar>
\ No newline at end of file
-- 
cgit v1.2.3