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

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

Issue 2834453007: Download Home : Remove drawer UI (Closed)
Patch Set: changes for tablets Created 3 years, 8 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_main.xml
diff --git a/chrome/android/java/res/layout/download_main.xml b/chrome/android/java/res/layout/download_main.xml
index 5e22ea6ac8b4810a45ddc6d8791d69be14bbd120..25823a8a35ab73927847fc416152c613f99803a9 100644
--- a/chrome/android/java/res/layout/download_main.xml
+++ b/chrome/android/java/res/layout/download_main.xml
@@ -3,39 +3,9 @@
Use of this source code is governed by a BSD-style license that can be
found in the LICENSE file. -->
-<android.support.v4.widget.DrawerLayout
+<org.chromium.chrome.browser.widget.selection.SelectableListLayout
xmlns:android="http://schemas.android.com/apk/res/android"
+ android:id="@+id/selectable_list"
android:layout_width="match_parent"
- android:layout_height="match_parent" >
-
- <!-- MAIN CONTENT -->
- <org.chromium.chrome.browser.widget.selection.SelectableListLayout
- android:id="@+id/selectable_list"
- android:layout_width="match_parent"
- android:layout_height="match_parent"
- android:background="@color/default_primary_color" />
-
- <!-- NAVIGATION DRAWER
- We can't assign a paddingStart or paddingEnd because the section highlights are full-bleed.
- -->
- <LinearLayout
- android:id="@+id/drawer_layout"
- android:layout_width="@dimen/drawer_width"
- android:layout_height="match_parent"
- android:layout_gravity="start"
- android:orientation="vertical"
- android:background="@android:color/white"
- android:clickable="true" >
-
- <include layout="@layout/download_manager_ui_space_widget"/>
-
- <ListView
- android:id="@+id/section_list"
- android:layout_width="match_parent"
- android:layout_height="0dp"
- android:layout_weight="1"
- android:divider="@null"
- android:dividerHeight="0dp" />
- </LinearLayout>
-
-</android.support.v4.widget.DrawerLayout>
+ android:layout_height="match_parent"
+ android:background="@color/appbar_background"/>
Theresa 2017/04/24 20:48:17 The background color should just be primary_defaul
shaktisahu 2017/04/24 22:07:09 Done.

Powered by Google App Engine
This is Rietveld 408576698