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

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

Issue 2143303002: [Downloads] Expose basic download history to UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleaning up the interface Created 4 years, 5 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/download_history_adapter_view.xml
diff --git a/chrome/android/java/res/layout/download_history_adapter_view.xml b/chrome/android/java/res/layout/download_history_adapter_view.xml
new file mode 100644
index 0000000000000000000000000000000000000000..875d0866d7a819c173b6bbd874da0991803f582a
--- /dev/null
+++ b/chrome/android/java/res/layout/download_history_adapter_view.xml
@@ -0,0 +1,62 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2016 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 a single item in the DownloadHistoryAdapterView. -->
+<RelativeLayout
+ xmlns:android="http://schemas.android.com/apk/res/android"
Ian Wen 2016/07/14 18:16:25 Possibly simplify our namings? Such as download_it
gone 2016/07/14 19:11:56 Eh, I'm leaning towards keeping it this way becaus
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content"
+ android:layout_marginStart="16dp"
+ android:layout_marginEnd="16dp"
+ android:layout_marginTop="16dp"
+ android:layout_marginBottom="16dp" >
+
+ <ImageView
+ android:id="@+id/icon_view"
+ android:layout_width="48dp"
+ android:layout_height="48dp"
+ android:layout_alignParentStart="true"
+ android:layout_alignParentTop="true"
+ android:layout_alignParentBottom="true"
+ android:layout_marginEnd="16dp"
+ android:scaleType="centerInside"
+ android:background="@color/light_active_color" />
+
+ <TextView
+ android:id="@+id/filename_view"
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content"
+ android:layout_alignParentTop="true"
+ android:layout_alignParentEnd="true"
+ android:layout_toEndOf="@+id/icon_view"
+ android:singleLine="true"
+ android:textColor="@color/default_text_color"
+ android:textSize="16sp" />
+
+ <TextView
+ android:id="@+id/hostname_view"
+ android:layout_width="0dp"
+ android:layout_height="wrap_content"
+ android:layout_alignParentBottom="true"
+ android:layout_toEndOf="@+id/icon_view"
+ android:layout_toStartOf="@+id/filesize_view"
+ android:layout_below="@+id/filename_view"
+ android:layout_marginEnd="16dp"
+ android:layout_marginTop="0dp"
+ android:ellipsize="start"
+ android:singleLine="true" />
+
+ <TextView
+ android:id="@+id/filesize_view"
+ android:layout_width="wrap_content"
+ android:layout_height="wrap_content"
+ android:layout_alignParentBottom="true"
+ android:layout_alignParentEnd="true"
+ android:layout_below="@+id/filename_view"
+ android:layout_marginTop="0dp"
+ android:textAlignment="viewEnd"
+ android:singleLine="true" />
+
+</RelativeLayout>
« no previous file with comments | « no previous file | chrome/android/java/res/layout/download_main.xml » ('j') | chrome/android/java/res/layout/download_main.xml » ('J')

Powered by Google App Engine
This is Rietveld 408576698