[Android] Make enhanced bookmark editing UI more straightforward.

chrome/android/java/res/layout/eb_edit.xml

 <?xml version="1.0" encoding="utf-8"?>
 <!--
      Copyright 2015 The Chromium Authors. All rights reserved.
      Use of this source code is governed by a BSD-style license that can be
      found in the LICENSE file.
 -->
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:chrome="http://schemas.android.com/apk/res-auto"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
@@ skipping 17 matching lines @@
 
         <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_marginEnd="16dp"
             android:layout_marginStart="16dp"
             android:layout_marginTop="8dp"
             android:layout_marginBottom="8dp"
             android:orientation="vertical" >
 
+            <!-- A trick to avoid focusing the below EditText initially. For some reason, setting

[Ian Wen, 2015/07/20 18:41:57]

Use SOFT_INPUT_STATE_HIDDEN instead. You can also configure it in xml.

[Kibeom Kim (inactive), 2015/07/20 20:04:47]

Yeah keyboard hiding is already configured in the xml. This removes focus. (it
can be focused without keyboard)

+                 android:focusableInTouchMode in the above LinearLayout or ScrollView didn't work
+                 in this context. -->
+            <View
+                android:layout_width="0dp"
+                android:layout_height="0dp"
+                android:focusable="true"
+                android:focusableInTouchMode="true" />
+
             <android.support.design.widget.TextInputLayout
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content" >
-
+                android:layout_height="wrap_content"
+                android:layout_marginTop="10dp"
+                android:layout_marginBottom="10dp" >

[Ian Wen, 2015/07/20 18:41:57]

When layouting edittexts, paddings are in general preferred, because it
increases the touch area and make it more convenient to edit. That was the
reason why I did not choose to use margin in the first place.

[Kibeom Kim (inactive), 2015/07/20 20:04:47]

Actually, that's the reason I used margin because I think we shouldn't enlarge
touch area unnecessarily. Do you know if there is a design guideline around
this? But anyways, I believe that TextInputLayout doesn't get touch-focus
itself.

                 <!-- android:text is set here is to avoid UI jumping. -->
                 <org.chromium.chrome.browser.widget.EmptyAlertEditText
                     android:id="@+id/title_text"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:hint="@string/bookmark_name"
                     android:text="@string/bookmark_name"
-                    android:imeOptions="flagNoExtractUi"
+                    android:imeOptions="flagNoExtractUi|actionDone"
                     android:inputType="textCapSentences|textAutoCorrect"
                     android:singleLine="true"
                     chrome:alertMessage="@string/bookmark_missing_title" />
             </android.support.design.widget.TextInputLayout>
 
             <TextView
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:paddingEnd="3dp"
-                android:paddingStart="3dp"
-                android:paddingTop="8dp"
+                android:layout_marginTop="10dp"
+                android:layout_marginBottom="8dp"
+                android:layout_marginStart="3dp"
+                android:layout_marginEnd="3dp"
                 android:text="@string/bookmark_folder"
-                android:textAppearance="@style/TextAppearance.AppCompat.Small" />
+                android:textAppearance="?android:attr/textAppearanceSmall"
+                android:textSize="12sp" />
 
             <TextView
                 android:id="@+id/folder_text"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
-                android:paddingBottom="8dp"
-                android:paddingEnd="3dp"
-                android:paddingStart="3dp"
-                android:paddingTop="8dp"
-                android:textAppearance="@style/TextAppearance.AppCompat.Medium"
+                android:layout_marginTop="8dp"
+                android:layout_marginBottom="12dp"
+                android:layout_marginStart="3dp"
+                android:layout_marginEnd="3dp"
+                android:textAppearance="?android:attr/textAppearanceMedium"
                 android:textColor="@color/dark_mode_tint" />
 
             <android.support.design.widget.TextInputLayout
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content" >
+                android:layout_height="wrap_content"
+                android:layout_marginTop="10dp"
+                android:layout_marginBottom="10dp" >
 
                 <org.chromium.chrome.browser.widget.EmptyAlertEditText
                     android:id="@+id/url_text"
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:hint="@string/bookmark_url"
                     android:text="@string/bookmark_url"
-                    android:imeOptions="flagNoExtractUi"
+                    android:imeOptions="flagNoExtractUi|actionDone"
                     android:inputType="textUri"
                     android:singleLine="true"
                     chrome:alertMessage="@string/bookmark_missing_url" />
             </android.support.design.widget.TextInputLayout>
         </LinearLayout>
     </ScrollView>
 </LinearLayout>