Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: chrome/android/java/res/layout/offline_download_header.xml

Issue 2670083002: [Download Home] Displaying offline page bundle per day (Closed)
Patch Set: Theresa's comments Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/res/layout/offline_download_header.xml
diff --git a/chrome/android/java/res/layout/offline_download_header.xml b/chrome/android/java/res/layout/offline_download_header.xml
new file mode 100644
index 0000000000000000000000000000000000000000..622e82611c748e285e92b6fefabc9ad4bbe3edc2
--- /dev/null
+++ b/chrome/android/java/res/layout/offline_download_header.xml
@@ -0,0 +1,56 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2017 The Chromium Authors. All rights reserved.
+ Use of this source code is governed by a BSD-style license that can be
+ found in the LICENSE file. -->
+
+<!-- Represents the offline pages section header in the DownloadHistoryAdapterView. -->
+<view class="org.chromium.chrome.browser.download.ui.OfflineGroupHeaderView"
+ xmlns:android="http://schemas.android.com/apk/res/android"
+ xmlns:chrome="http://schemas.android.com/apk/res-auto"
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content" >
+
+ <LinearLayout
+ android:id="@+id/layout_container"
+ style="@style/DownloadRowContainer" >
+
+ <org.chromium.chrome.browser.widget.TintedImageView
+ android:id="@+id/icon_view"
+ style="@style/DownloadIconView"
+ android:src="@drawable/ic_drive_site_white_24dp" />
+
+ <RelativeLayout
+ android:id="@+id/completed_layout"
+ android:layout_width="0dp"
+ android:layout_height="wrap_content"
+ android:layout_weight="1" >
+
+ <TextView
+ android:id="@+id/page_count_text"
+ style="@style/DownloadFileNameStyle" />
Theresa 2017/02/10 00:55:12 nit: maybe this style should be DownloadTitleStyle
shaktisahu 2017/02/10 21:30:24 Done.
+
+ <ImageView
+ android:id="@+id/expand_icon"
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content"
+ android:layout_alignParentTop="true"
+ android:layout_toEndOf="@+id/page_count_text"
+ android:layout_marginTop="0dp"
+ android:paddingEnd="16dp"
+ android:contentDescription="@null"
+ android:src="@drawable/ic_expanded" />
+
+ <TextView
+ android:id="@+id/description"
+ style="@style/DownloadHostNameStyle"
Theresa 2017/02/10 00:55:12 nit: maybe this style should be called DownloadDes
shaktisahu 2017/02/10 21:30:24 Done.
+ android:layout_toStartOf="@+id/filesize_view"
+ android:layout_below="@+id/page_count_text"
+ android:text="@string/download_manager_offline_header_description" />
+
+ <TextView
+ android:id="@+id/filesize_view"
+ style="@style/DownloadFileSizeStyle"
+ android:layout_below="@+id/page_count_text" />
+ </RelativeLayout>
+ </LinearLayout>
+</view>

Powered by Google App Engine
This is Rietveld 408576698