Restore the duplicate search form in the header

It turns out the duplicate form is used, for the the mobile view.

diff --git a/app/assets/javascripts/index.js b/app/assets/javascripts/index.js
index a300cc2807abe0f946c6654b3afd9dfb89d680e8..55e612f17668f6e3da055b2fdcf003b3a03c9285 100644 (file)
--- a/app/assets/javascripts/index.js
+++ b/app/assets/javascripts/index.js
@@ -229,7 +229,7 @@ $(document).ready(function () {
 
     page.load = function() {
       if (!("autofocus" in document.createElement("input"))) {
-        $(".search_form input[name=query]").focus();
+        $("#sidebar .search_form input[name=query]").focus();
       }
       return map.getState();
     };

diff --git a/app/views/layouts/_search.html.erb b/app/views/layouts/_search.html.erb
index 55ce54219eab18da9719f06686bf59aec44c9af4..732e57d31505f20f6382c40c7fe238c6fe4e8ea1 100644 (file)
--- a/app/views/layouts/_search.html.erb
+++ b/app/views/layouts/_search.html.erb
@@ -1,7 +1,7 @@
 <%= form_tag search_path, :class => "search_form" do %>
   <%= submit_tag t('site.search.submit_text') %>
   <div class='query_wrapper'>
-    <input type="text" name="query" value="<%= params[:query] %>" placeholder="<%= t('site.search.search') %>" autofocus>
+    <%= text_field_tag "query", params[:query], :placeholder => t("site.search.search"), :autofocus => autofocus %>
     <%= link_to t('site.search.where_am_i'), '#', { :class => "describe_location", :title => t('site.search.where_am_i_title') } %>
   </div>
 <% end %>

diff --git a/app/views/layouts/map.html.erb b/app/views/layouts/map.html.erb
index ab2d0f77ac449a454fc7c2eb08b4c777f6f4d835..e72214cbefe86fae1d9e6c816bd99766fec39c3c 100644 (file)
--- a/app/views/layouts/map.html.erb
+++ b/app/views/layouts/map.html.erb
@@ -16,9 +16,13 @@
   <% end %>
 <% end %>
 
+<% content_for :header do %>
+  <%= render :partial => "layouts/search", :locals => { :autofocus => false } %>
+<% end %>
+
 <% content_for :content do %>
   <div id="sidebar">
-    <%= render :partial => "layouts/search" %>
+    <%= render :partial => "layouts/search", :locals => { :autofocus => true } %>
 
     <% if STATUS == :database_offline or STATUS == :api_offline %>
       <p class="error"><%= t 'layouts.osm_offline' %></p>