Chromium Code Reviews| Index: chrome/android/java/res/layout/autofill_credit_card_editor.xml |
| diff --git a/chrome/android/java/res/layout/autofill_credit_card_editor.xml b/chrome/android/java/res/layout/autofill_credit_card_editor.xml |
| index 19fd877105542e0a90f935d0ee3bde334cf0793b..1dd466293749a5cc2b12103c366fabc26cb32f4e 100644 |
| --- a/chrome/android/java/res/layout/autofill_credit_card_editor.xml |
| +++ b/chrome/android/java/res/layout/autofill_credit_card_editor.xml |
| @@ -10,6 +10,60 @@ |
| xmlns:android="http://schemas.android.com/apk/res/android" |
| xmlns:app="http://schemas.android.com/apk/res-auto" > |
| + <!-- The section that's shown only for server cards. --> |
|
gone
2016/06/29 20:09:41
One big problem with jamming in the controls into
please use gerrit instead
2016/06/30 01:19:27
Layouts are split-up.
|
| + <org.chromium.ui.widget.TextViewWithClickableSpans |
| + android:id="@+id/server_card_label" |
| + android:layout_width="wrap_content" |
|
gone
2016/06/29 20:09:41
match_parent
please use gerrit instead
2016/06/30 01:19:27
Done.
|
| + android:layout_height="wrap_content" |
| + style="@style/PreferenceTitle" |
| + android:text="Visa 3832" /> |
| + |
| + <org.chromium.ui.widget.TextViewWithClickableSpans |
| + android:id="@+id/server_card_sublabel" |
| + android:layout_width="wrap_content" |
|
gone
2016/06/29 20:09:42
match_parent
please use gerrit instead
2016/06/30 01:19:27
Done.
|
| + android:layout_height="wrap_content" |
| + style="@style/PreferenceSummary" |
| + android:text="08/2017" /> |
| + |
| + <LinearLayout |
| + android:id="@+id/server_card_edit_link_container" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:paddingTop="@dimen/pref_autofill_field_top_padding" |
| + android:paddingBottom="@dimen/pref_autofill_field_top_padding" |
| + android:orientation="horizontal" |
| + android:baselineAligned="false" > |
| + |
| + <org.chromium.ui.widget.TextViewWithClickableSpans |
| + android:layout_width="wrap_content" |
| + android:layout_height="wrap_content" |
| + android:paddingEnd="15dp" |
| + android:layout_weight="1" |
| + style="@style/PreferenceTitle" |
| + android:text="@string/autofill_from_google_account_long" /> |
| + |
| + <org.chromium.ui.widget.TextViewWithClickableSpans |
| + android:id="@+id/server_card_edit_link" |
| + android:layout_width="wrap_content" |
| + android:layout_height="wrap_content" |
| + android:textColor="?android:attr/textColorLink" |
| + android:textAllCaps="true" |
| + android:text="@string/autofill_wallet_management_link_text" /> |
| + |
| + </LinearLayout> |
| + |
| + <org.chromium.ui.widget.ButtonCompat |
|
gone
2016/06/29 20:09:41
This button is just kind of floating here randomly
please use gerrit instead
2016/06/30 01:19:27
There was a missing label. Added it back.
|
| + android:id="@+id/server_card_clear_button" |
| + android:layout_height="wrap_content" |
| + android:layout_width="wrap_content" |
| + android:paddingTop="@dimen/pref_autofill_field_top_padding" |
|
gone
2016/06/29 20:09:41
Don't customize the padding of the button. Are yo
please use gerrit instead
2016/06/30 01:19:27
Done.
|
| + android:paddingBottom="@dimen/pref_autofill_field_top_padding" |
| + android:minHeight="40dp" |
|
gone
2016/06/29 20:09:41
Don't set the minHeight; should be handled by the
please use gerrit instead
2016/06/30 01:19:27
Done.
|
| + android:textColor="#fff" |
| + android:text="@string/autofill_clear_local_copy_button" |
| + app:buttonColor="@color/pref_accent_color" /> |
| + |
| + <!-- The section that's shown only for local cards. --> |
| <org.chromium.chrome.browser.widget.FloatLabelLayout |
| android:id="@+id/credit_card_name_label" |
| android:layout_width="match_parent" |
| @@ -38,6 +92,7 @@ |
| </org.chromium.chrome.browser.widget.FloatLabelLayout> |
| <TextView |
| + android:id="@+id/credit_card_expiration_label" |
| android:layout_width="match_parent" |
| android:layout_height="wrap_content" |
| android:focusable="true" |
| @@ -48,6 +103,7 @@ |
| android:paddingEnd="@dimen/pref_autofill_field_horizontal_padding" /> |
| <LinearLayout |
| + android:id="@+id/credit_card_expiration_container" |
| android:layout_width="match_parent" |
| android:layout_height="wrap_content" |
| android:orientation="horizontal" |
| @@ -82,6 +138,7 @@ |
| </LinearLayout> |
| </LinearLayout> |
| + <!-- The section that's shown for both local and server cards. --> |
| <TextView |
| android:layout_width="match_parent" |
| android:layout_height="wrap_content" |