chrome/android/java/res/layout/autofill_text.xml - Issue 16994006: [Android] Change the AutofillPopup UI.

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

Issue 16994006: [Android] Change the AutofillPopup UI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Adding a hack to prevent the popup from expanding on top of the keyboard Created 7 years, 6 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

« no previous file with comments | « chrome/android/java/res/drawable/autofill_popup_background_up.9.png ('k') | chrome/android/java/res/values-v17/styles.xml » ('j') | chrome/android/java/src/org/chromium/chrome/browser/autofill/AutofillPopup.java » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/android/java/res/layout/autofill_text.xml

diff --git a/chrome/android/java/res/layout/autofill_text.xml b/chrome/android/java/res/layout/autofill_text.xml

index 08aede0fdd9831e96e4ce4006480b88e52d10310..0ded2a24a6b97d148176c9f054735def8a86ff57 100644

--- a/chrome/android/java/res/layout/autofill_text.xml

+++ b/chrome/android/java/res/layout/autofill_text.xml

@@ -5,28 +5,29 @@

found in the LICENSE file.

-->

-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"

- android:layout_height="fill_parent"

- android:layout_width="fill_parent"

- android:background="#FFF"

- android:orientation="horizontal">

+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

+ android:id="@+id/autofill_menu_text"

+ android:layout_width="fill_parent"

+ android:layout_height="48dp"

+ android:minHeight="48dp"

+ android:orientation="vertical"

+ android:gravity="center_vertical">

<TextView android:id="@+id/autofill_label"

- android:layout_alignParentEnd="true"

android:layout_width="wrap_content"

- android:layout_height="fill_parent"

- android:padding="5dp"

- android:textSize="18sp"

- android:textColor="#CCC"

+ android:layout_height="wrap_content"

android:ellipsize="end"

- android:singleLine="true"/>

- <TextView android:id="@+id/autofill_name"

- android:layout_alignParentStart="true"

- android:layout_width="wrap_content"

- android:layout_height="fill_parent"

- android:padding="5dp"

- android:background="#FFF"

+ android:singleLine="true"

android:textSize="18sp"

- android:textColor="#000"

+ android:layout_marginStart="10dp"

+ android:layout_marginEnd="10dp"/>

+ <TextView android:id="@+id/autofill_sublabel"

+ android:layout_width="wrap_content"

+ android:layout_height="wrap_content"

+ android:textSize="14sp"

+ android:textColor="#8b8b8b"

android:ellipsize="end"

- android:singleLine="true"/>

-</RelativeLayout>

+ android:singleLine="true"

+ android:includeFontPadding="false"

Ted C 2013/06/14 20:57:53 Why add this out of curiosity?

aurimas (slooooooooow) 2013/06/14 21:09:09 This seems to reduce the padding between label and

+ android:layout_marginStart="10dp"

+ android:layout_marginEnd="10dp"/>

+</LinearLayout>