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

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

Issue 1780893005: Visual refresh for bookmarks widget. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 4 years, 9 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/bookmark_thumbnail_widget_item.xml
diff --git a/chrome/android/java/res/layout/bookmark_thumbnail_widget_item.xml b/chrome/android/java/res/layout/bookmark_thumbnail_widget_item.xml
index 5cb190156c5e7b1cd36e7323eaed5668d3c09e36..daa49355396f55ee6b08acb6465fa43fa6fcdca3 100644
--- a/chrome/android/java/res/layout/bookmark_thumbnail_widget_item.xml
+++ b/chrome/android/java/res/layout/bookmark_thumbnail_widget_item.xml
@@ -3,53 +3,31 @@
Use of this source code is governed by a BSD-style license that can be
found in the LICENSE file. -->
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:id="@+id/list_item"
android:layout_width="match_parent"
- android:layout_height="wrap_content" >
+ android:layout_height="wrap_content"
+ android:gravity="center_vertical"
+ android:orientation="horizontal" >
<ImageView
- android:id="@+id/thumb"
- android:layout_width="match_parent"
- android:layout_height="@dimen/widget_thumbnail_height"
- android:background="@drawable/border_thumb_bookmarks_widget_holo"
- android:cropToPadding="true"
- android:scaleType="centerCrop"
- android:src="@drawable/thumbnail_bookmarks_widget_no_bookmark_holo"
- android:contentDescription="@null" />
+ android:id="@+id/favicon"
+ android:layout_width="64dp"
+ android:layout_height="48dp"
+ android:contentDescription="@null"
+ android:scaleType="center" />
- <LinearLayout
- android:layout_width="match_parent"
+ <TextView
+ android:id="@+id/title"
+ android:layout_width="0dp"
android:layout_height="wrap_content"
- android:layout_alignBottom="@id/thumb"
- android:layout_marginBottom="2dp"
- android:layout_marginEnd="2dp"
- android:layout_marginStart="2dp"
- android:background="@drawable/overlay_url_bookmark_widget_holo" >
+ android:layout_weight="1"
+ android:gravity="center_vertical"
+ android:minHeight="48dp"
+ android:paddingEnd="16dp"
+ android:singleLine="true"
+ android:textAlignment="viewStart"
+ android:textColor="@color/default_text_color"
+ android:textSize="16sp" />
- <ImageView
- android:id="@+id/favicon"
- android:layout_width="@dimen/widget_favicon_size"
- android:layout_height="@dimen/widget_favicon_size"
- android:layout_gravity="center_vertical"
- android:layout_marginBottom="4dp"
- android:layout_marginEnd="8dp"
- android:layout_marginStart="4dp"
- android:layout_marginTop="4dp"
- android:scaleType="centerInside"
- android:src="@drawable/globe_incognito_favicon"
- android:contentDescription="@null" />
-
- <TextView
- android:id="@+id/label"
- android:layout_width="match_parent"
- android:layout_height="wrap_content"
- android:layout_gravity="center_vertical"
- android:ellipsize="end"
- android:singleLine="true"
- android:textColor="@android:color/black"
- android:textSize="12sp"
- android:typeface="sans" />
- </LinearLayout>
-
-</RelativeLayout>
+</LinearLayout>

Powered by Google App Engine
This is Rietveld 408576698