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

Unified Diff: chrome/android/java/res/values/dimens.xml

Issue 177863008: Update the AppBannerView appearance again (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing comments hopefully Created 6 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/values/dimens.xml
diff --git a/chrome/android/java/res/values/dimens.xml b/chrome/android/java/res/values/dimens.xml
index 13cfde5bbd1aa0b1339fddf87723ccb4d7bb2019..887b0817b492dc5fd4d8f936c0f9c5112e919248 100644
--- a/chrome/android/java/res/values/dimens.xml
+++ b/chrome/android/java/res/values/dimens.xml
@@ -43,7 +43,7 @@
<dimen name="app_banner_margin_sides">8dp</dimen>
<dimen name="app_banner_margin_bottom">8dp</dimen>
<dimen name="app_banner_padding">8dp</dimen>
- <dimen name="app_banner_padding_content">4dp</dimen>
+ <dimen name="app_banner_padding_controls">4dp</dimen>
<dimen name="app_banner_button_text_size">14sp</dimen>
<dimen name="app_banner_button_height">28dp</dimen>
@@ -59,8 +59,9 @@
<dimen name="app_banner_title_margin_top">0dp</dimen>
<dimen name="app_banner_title_margin_bottom">8dp</dimen>
- <dimen name="app_banner_logo_height">22dp</dimen>
- <dimen name="app_banner_logo_margin_bottom">3dp</dimen>
+ <dimen name="app_banner_logo_height">24dp</dimen>
+ <dimen name="app_banner_logo_margin_top">4dp</dimen>
+ <dimen name="app_banner_logo_margin_bottom">0dp</dimen>
<dimen name="app_banner_star_height">16dp</dimen>
</resources>

Powered by Google App Engine
This is Rietveld 408576698