commit 8844d19f6879802ba02463df6cf2b8bfbaf68b0a
author Steve Block <steveblock@google.com> Tue Sep 29 13:14:41 2009 +0100
committer Steve Block <steveblock@google.com> Tue Sep 29 19:10:44 2009 +0100
tree 89040a6aaefe6acabc61ee36b231b3ccff6b6f11
parent c46f828c17e8f171aa62910cba7ba6af6f7302ae

Makes Geolocation permissions prompt pretty.

Change-Id: I237fc0845de2297babc7da448a2199c5344e2e83

res/layout/geolocation_permissions_prompt.xml

diff --git a/res/layout/geolocation_permissions_prompt.xml b/res/layout/geolocation_permissions_prompt.xml
index 404637b..3d8e230 100755
--- a/res/layout/geolocation_permissions_prompt.xml
+++ b/res/layout/geolocation_permissions_prompt.xml
@@ -14,7 +14,6 @@
      limitations under the License.
 
      This is the layout for the Geolocation permissions prompt.
-     FIXME: Make this looks pretty.
 -->
 
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
@@ -28,6 +27,7 @@
         android:layout_width="fill_parent"
         android:layout_height="wrap_content"
         android:background="@color/geolocation_permissions_prompt_background"
+        android:padding="6dip"
         android:visibility="gone">
 
         <!-- 'google.com wants to know your location' -->
@@ -35,7 +35,15 @@
             android:layout_width="fill_parent"
             android:layout_height="wrap_content"
             android:textSize="14dip"
-            android:textColor="@color/white" />
+            android:textColor="@color/black" />
+
+        <!-- Checkbox -->
+        <CheckBox android:id="@+id/remember"
+            android:text="@string/geolocation_permissions_prompt_remember"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:textSize="14dip"
+            android:textColor="@color/black" />
 
         <!-- Buttons -->
         <LinearLayout
@@ -43,21 +51,15 @@
             android:layout_height="wrap_content">
             <Button android:id="@+id/share_button"
                 android:text="@string/geolocation_permissions_prompt_share"
-                android:layout_width="wrap_content"
+                android:layout_weight="1"
+                android:layout_width="0dip"
                 android:layout_height="wrap_content" />
             <Button android:id="@+id/dont_share_button"
                 android:text="@string/geolocation_permissions_prompt_dont_share"
-                android:layout_width="wrap_content"
+                android:layout_weight="1"
+                android:layout_width="0dip"
                 android:layout_height="wrap_content" />
         </LinearLayout>
 
-        <CheckBox android:id="@+id/remember"
-            android:text="@string/geolocation_permissions_prompt_remember"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:drawablePadding="10dip"
-            android:textSize="16dip"
-            android:textColor="@color/white" />
-
     </LinearLayout>
 </LinearLayout>

res/layout/tab.xml

diff --git a/res/layout/tab.xml b/res/layout/tab.xml
index ea9cd1a..e6ec970 100755
--- a/res/layout/tab.xml
+++ b/res/layout/tab.xml
@@ -19,7 +19,8 @@
      Currently, the only such element is the Geolocation permissions prompt.
 -->
 
-<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical"
     android:fitsSystemWindows="true"
     android:layout_width="fill_parent"
     android:layout_height="fill_parent">
@@ -27,7 +28,8 @@
     <!-- Wrapper layout for the WebView, which must be in a FrameLayout. -->
     <FrameLayout android:id="@+id/webview_wrapper"
         android:layout_width="fill_parent"
-        android:layout_height="fill_parent" />
+        android:layout_height="0dip"
+        android:layout_weight="1" />
 
     <!-- Geolocation permissions prompt -->
     <com.android.browser.GeolocationPermissionsPrompt
@@ -35,4 +37,4 @@
         android:layout_width="fill_parent"
         android:layout_height="wrap_content" />
 
-</FrameLayout>
+</LinearLayout>