Chromium Code Reviews| Index: chrome/android/java/res/layout/autofill_profile_editor.xml |
| diff --git a/chrome/android/java/res/layout/autofill_profile_editor.xml b/chrome/android/java/res/layout/autofill_profile_editor.xml |
| index 6c26d133ff5c4866f9b6f3f1ae5d66f6d03c34e0..7c4d4033d15d139616d56268c0feb1e88a5d85be 100644 |
| --- a/chrome/android/java/res/layout/autofill_profile_editor.xml |
| +++ b/chrome/android/java/res/layout/autofill_profile_editor.xml |
| @@ -4,6 +4,7 @@ |
| found in the LICENSE file. --> |
| <ScrollView xmlns:android="http://schemas.android.com/apk/res/android" |
| + xmlns:app="http://schemas.android.com/apk/res-auto" |
| android:layout_width="match_parent" |
| android:layout_height="match_parent" |
| android:fillViewport="true" > |
| @@ -21,6 +22,16 @@ |
| android:orientation="vertical"> |
| <!-- Editable fields for the profile --> |
| + <TextView |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:textAppearance="@style/PreferenceFloatLabelTextAppearance" |
| + android:text="@string/autofill_profile_editor_country" /> |
| + <Spinner |
| + android:id="@+id/countries" |
| + android:layout_width="fill_parent" |
| + android:layout_height="wrap_content" |
| + android:contentDescription="@string/autofill_profile_editor_country" /> |
| <LinearLayout |
| android:id="@+id/autofill_profile_widget_root" |
| @@ -29,29 +40,33 @@ |
| android:orientation="vertical" > |
| </LinearLayout> |
| - <TextView |
| - style="@style/BoldTextFieldLabel" |
| - android:text="@string/autofill_profile_editor_phone_number" /> |
| - |
| - <EditText |
| - android:id="@+id/autofill_profile_editor_phone_number_edit" |
| + <org.chromium.chrome.browser.widget.FloatLabelLayout |
| + android:id="@+id/autofill_profile_editor_phone_number_label" |
|
newt (away)
2015/01/28 20:20:18
Shorter IDs are OK too, e.g. autofill_phone_number
Theresa
2015/01/29 05:22:13
Done.
|
| android:layout_width="match_parent" |
| - android:layout_height="wrap_content" |
| - android:imeOptions="flagNoExtractUi" |
| - android:inputType="phone" |
| - android:singleLine="true" /> |
| + android:layout_height="wrap_content" > |
| + <EditText |
|
newt (away)
2015/01/28 20:20:18
So, do you need to set the content description her
Theresa
2015/01/29 05:22:13
Yes, done.
|
| + android:id="@+id/autofill_profile_editor_phone_number_edit" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:imeOptions="flagNoExtractUi" |
| + android:inputType="phone" |
| + android:singleLine="true" |
| + android:hint="@string/autofill_profile_editor_phone_number" /> |
| + </org.chromium.chrome.browser.widget.FloatLabelLayout> |
| - <TextView |
| - style="@style/BoldTextFieldLabel" |
| - android:text="@string/autofill_profile_editor_email_address" /> |
| - |
| - <EditText |
| - android:id="@+id/autofill_profile_editor_email_address_edit" |
| + <org.chromium.chrome.browser.widget.FloatLabelLayout |
| + android:id="@+id/autofill_profile_editor_email_address_label" |
| android:layout_width="match_parent" |
| - android:layout_height="wrap_content" |
| - android:imeOptions="flagNoExtractUi" |
| - android:inputType="textEmailAddress" |
| - android:singleLine="true" /> |
| + android:layout_height="wrap_content" > |
| + <EditText |
| + android:id="@+id/autofill_profile_editor_email_address_edit" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:imeOptions="flagNoExtractUi" |
| + android:inputType="textEmailAddress" |
| + android:singleLine="true" |
| + android:hint="@string/autofill_profile_editor_email_address" /> |
| + </org.chromium.chrome.browser.widget.FloatLabelLayout> |
| </LinearLayout> |