Commit c33da1f7 authored by Lakshmi Kumar Dabbiru's avatar Lakshmi Kumar Dabbiru Committed by Commit Bot

Dont make EditText Focusable by default

EditText should not be in focus by default and so move focus to layout instead of edit text.
This makes *hint* grey by default and turns blue when user starts to edit.

UX implementation review deck that has all screenshots

https://docs.google.com/presentation/d/1OTWsG8au8qS7FhcXyvvbOk5fhEmGE5YEEzSGgq9MVKw/edit?ts=5c2ea8b9#slide=id.g404c5faffe_1_12

Bug: 913646
Change-Id: Ib78771c0a4cdd884505d40f8d723999d91207123
Reviewed-on: https://chromium-review.googlesource.com/c/1395560
Commit-Queue: Lakshmi Kumar Dabbiru <dlkumar@google.com>
Reviewed-by: default avatarBecky Zhou <huayinz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#620018}
parent a31c6b96
......@@ -10,6 +10,8 @@
android:layout_height="wrap_content"
style="@style/AlertDialogContent"
android:minHeight="36dp"
android:focusable="true"
android:focusableInTouchMode="true"
android:layout_marginBottom="32dp"
android:paddingBottom="16dp"
android:layout_marginTop="6dp"
......@@ -19,7 +21,6 @@
<org.chromium.chrome.browser.widget.CompatibilityTextInputLayout
android:id="@+id/cc_name"
android:labelFor="@+id/cc_name_edit"
android:focusableInTouchMode="true"
android:layout_width="0dp"
android:layout_weight="1"
android:layout_height="wrap_content">
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment