From: Tom Hughes <tom@compton.nu>
Date: Mon, 5 May 2008 16:39:33 +0000 (+0000)
Subject: Rework the new Go button on the search form so that it actually stands
X-Git-Tag: live~8755
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/feb47bc7396f02f35dbadc42b09288f0af004fe8?ds=sidebyside

Rework the new Go button on the search form so that it actually stands
some chance of rendering as intended.

It's still not really ideal to steal space from the seach box of course...

Closes #589.
---

diff --git a/app/views/site/_search.rhtml b/app/views/site/_search.rhtml
index 43853dcf0..f18587f73 100644
--- a/app/views/site/_search.rhtml
+++ b/app/views/site/_search.rhtml
@@ -38,10 +38,14 @@
     <% form_remote_tag(:loading => "startSearch()",
                        :complete => "endSearch()",
                        :url => { :controller => :geocoder, :action => :search }) do %>
-      <%= text_field_tag :query, h(params[:query]) %>
-  
-      <%= submit_tag 'Go' %></td>
-      <% end %>
+      <table>
+        <tr>
+          <td><%= text_field_tag :query, h(params[:query]) %></td>
+          <td></td>
+          <td><%= submit_tag 'Go' %></td>
+        </tr>
+      </table>
+    <% end %>
     </span>
     <p id="search_active">Searching...</p>
     </div>
diff --git a/public/stylesheets/site.css b/public/stylesheets/site.css
index e1b877e03..2ede490ef 100644
--- a/public/stylesheets/site.css
+++ b/public/stylesheets/site.css
@@ -354,14 +354,18 @@ hides rule from IE5-Mac \*/
   padding: 0px;
 }
 
+.optionalbox input {
+  width: 100%;
+}
 
-.optionalbox input[type="text"] {
-  width: 110px;
+.optionalbox td {
+  margin: 0px;
+  padding: 0px;
 }
 
 .search_form {
   height: 16px;
-  padding-bottom: 2px;
+  padding-bottom: 6px;
 }
 
 #search_active {