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

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

Issue 1430273002: [Smart Lock, UI] Add ability to have an icon to the SnanackBar. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments. Created 5 years, 1 month 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/snackbar.xml
diff --git a/chrome/android/java/res/layout/snackbar.xml b/chrome/android/java/res/layout/snackbar.xml
index 119916007b5f9f69e16eb9e9089d439bc6cc50a6..68a3e10621e688e660c1f7b7f6ede000dcfaecd3 100644
--- a/chrome/android/java/res/layout/snackbar.xml
+++ b/chrome/android/java/res/layout/snackbar.xml
@@ -9,6 +9,13 @@
android:minHeight="@dimen/snackbar_min_height"
android:orientation="horizontal" >
+ <ImageView android:id="@+id/snackbar_icon"
+ android:layout_width="24dp"
gone 2015/11/10 00:35:04 nit: indentation is messed up
+ android:layout_height="24dp"
+ android:layout_gravity="center_vertical"
+ android:layout_marginStart="12dp"
gone 2015/11/10 00:35:04 can you explicitly state what ImageView.ScaleType
+ android:contentDescription="@null"/>
+
<org.chromium.chrome.browser.snackbar.TemplatePreservingTextView
android:id="@+id/snackbar_message"
android:layout_width="0dp"

Powered by Google App Engine
This is Rietveld 408576698