commit 75630678154c89662b7f8e5fb19d0b77a2d7abb8
author Leon Scroggins <scroggo@google.com> Thu Jan 13 17:56:15 2011 -0500
committer Leon Scroggins <scroggo@google.com> Thu Jan 13 18:30:03 2011 -0500
tree 8371174e699e0e70931a482d74b2939c32b2f994
parent 58fddd3154418fbdf7560db5662fcf6d248061e3

Show a remove button for existing bookmarks.

Bug:3222677

Change-Id: Id11585bc68f239a02aa477898f6c00792ad44093

res/layout/browser_add_bookmark.xml

diff --git a/res/layout/browser_add_bookmark.xml b/res/layout/browser_add_bookmark.xml
index e25f903..5eebc0b 100644
--- a/res/layout/browser_add_bookmark.xml
+++ b/res/layout/browser_add_bookmark.xml
@@ -25,6 +25,8 @@
         android:minHeight="?android:attr/listPreferredItemHeight"
         android:gravity="center_vertical"
         android:visibility="gone"
+        android:paddingLeft="5dip"
+        android:paddingRight="5dip"
         >
         <com.android.browser.BreadCrumbView android:id="@+id/crumbs"
             android:layout_width="wrap_content"
@@ -53,16 +55,41 @@
             android:layout_centerVertical="true"
             />
     </RelativeLayout>
-    <TextView android:id="@+id/fake_title"
-        android:layout_width="wrap_content"
+    <LinearLayout android:id="@+id/title_holder"
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:orientation="horizontal"
         android:minHeight="?android:attr/listPreferredItemHeight"
-        android:drawableLeft="@drawable/ic_list_bookmark"
-        android:text="@string/bookmark_this_page"
-        android:layout_gravity="left"
-        android:gravity="center_vertical"
-        android:textAppearance="?android:attr/textAppearanceMedium" />
-
+        android:paddingLeft="5dip"
+        android:paddingRight="5dip"
+        >
+        <TextView android:id="@+id/fake_title"
+            android:layout_width="0dip"
+            android:layout_weight="1"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:gravity="center_vertical"
+            android:drawableLeft="@drawable/ic_list_bookmark"
+            android:text="@string/bookmark_this_page"
+            android:textAppearance="?android:attr/textAppearanceMedium" />
+        <ImageView android:id="@+id/remove_divider"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:src="@drawable/crumb_divider"
+            android:layout_centerVertical="true"
+            android:visibility="gone"
+            />
+        <TextView android:id="@+id/remove"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:gravity="center_vertical"
+            android:text="@string/remove"
+            android:drawableLeft="@*android:drawable/ic_menu_delete"
+            android:visibility="gone"
+            android:textAppearance="?android:attr/textAppearanceMedium" />
+    </LinearLayout>
     <View android:id="@+id/titleDivider"
         android:layout_width="match_parent"
         android:layout_height="1dip"