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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java

Issue 2450113003: [Payments] Increase spacing for payment request credit card and address editors (Closed)
Patch Set: for better indentation Created 4 years, 2 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/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java
index cc531029429101ce7fe582608b94a9e8fc912306..f41ea7c7dc84258d35270463216f884adc9a61c5 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/EditorIconsField.java
@@ -13,6 +13,7 @@ import android.widget.ImageView;
import android.widget.LinearLayout;
import android.widget.TextView;
+import org.chromium.base.ApiCompatibilityUtils;
import org.chromium.chrome.R;
/**
@@ -39,15 +40,21 @@ class EditorIconsField {
LinearLayout container = (LinearLayout) mLayout.findViewById(R.id.icons_container);
int size = context.getResources().getDimensionPixelSize(
R.dimen.payments_editor_icon_list_size);
+ int margin = context.getResources().getDimensionPixelSize(
+ R.dimen.payments_section_small_spacing);
+ LinearLayout.LayoutParams layoutParams =
+ new LinearLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
+ ApiCompatibilityUtils.setMarginEnd(layoutParams, margin);
for (int i = 0; i < fieldModel.getIconResourceIds().size(); i++) {
ImageView icon = new ImageView(context);
icon.setImageResource(fieldModel.getIconResourceIds().get(i));
+ icon.setBackgroundResource(R.drawable.payments_ui_logo_bg);
icon.setContentDescription(context.getString(
fieldModel.getIconDescriptionsForAccessibility().get(i)));
icon.setAdjustViewBounds(true);
icon.setMaxWidth(size);
icon.setMaxHeight(size);
- container.addView(icon, LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
+ container.addView(icon, layoutParams);
}
}

Powered by Google App Engine
This is Rietveld 408576698