From: Tom MacWright <tom@macwright.org>
Date: Wed, 11 Apr 2012 16:14:28 +0000 (-0400)
Subject: This simply moves GPS Traces to the left bar
X-Git-Tag: live~5551
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/dd3b577bd5bbe4f9d9d1cac8e0f328a16b75f7f5?ds=sidebyside

This simply moves GPS Traces to the left bar

The left bar is a bit of a kludge of random items right now, but it's a
little better having the GPS Traces item there, because it now makes the
top bar only items that are actually related to the map state.
---

diff --git a/app/views/layouts/site.html.erb b/app/views/layouts/site.html.erb
index c572b1926..d25360f1b 100644
--- a/app/views/layouts/site.html.erb
+++ b/app/views/layouts/site.html.erb
@@ -31,18 +31,15 @@
         editclass = 'geolink llz object minzoom13 disabled'
         historyclass = 'geolink bbox minzoom11'
         exportclass = 'geolink llz layers'
-        traceclass = ''
-        viewclass += ' active' if params['controller'] == 'site' and params['action'] == 'index' 
-        editclass += ' active' if params['controller'] == 'site' and params['action'] == 'edit' 
-        historyclass += ' active' if params['controller'] == 'changeset' and params['action'] == 'list' 
+        viewclass += ' active' if params['controller'] == 'site' and params['action'] == 'index'
+        editclass += ' active' if params['controller'] == 'site' and params['action'] == 'edit'
+        historyclass += ' active' if params['controller'] == 'changeset' and params['action'] == 'list'
         exportclass += ' active' if params['controller'] == 'site' and params['action'] == 'export'
-        traceclass += ' active' if params['controller'] == 'trace'
         %>
         <li><%= link_to t('layouts.view'), {:controller => 'site', :action => 'index'}, {:id => 'viewanchor', :title => t('layouts.view_tooltip'), :class => viewclass} %></li>
         <li><%= link_to h(t('layouts.edit')) + '&nbsp;&#x25be;'.html_safe, {:controller => 'site', :action => 'edit'}, {:id => 'editanchor', :title => t('javascripts.site.edit_tooltip'), :class => editclass} %></li>
         <li><%= link_to t('layouts.history'), {:controller => 'changeset', :action => 'list' }, {:id => 'historyanchor', :title => t('javascripts.site.history_tooltip'), :class => historyclass} %></li>
         <li><%= link_to t('layouts.export'), {:controller => 'site', :action => 'export'}, {:id => 'exportanchor', :title => t('layouts.export_tooltip'), :class => exportclass} %></li>
-        <li><%= link_to t('layouts.gps_traces'), {:controller => 'trace', :action => 'list', :display_name => nil, :tag => nil, :page => nil}, {:id => 'traceanchor', :title => t('layouts.gps_traces_tooltip'), :class => traceclass} %></li>
       </ul>
     </div>
 
@@ -105,6 +102,7 @@
       <div id="left_menu" class="left_menu">
         <ul>
           <%= yield :left_menu %>
+          <li><%= link_to t('layouts.gps_traces'), {:controller => 'trace', :action => 'list', :display_name => nil, :tag => nil, :page => nil}, {:id => 'traceanchor', :title => t('layouts.gps_traces_tooltip') } %></li>
           <li><%= t'layouts.help' %>
               <ul>
               <li><%= link_to(t('layouts.help_centre'), t('layouts.help_url'), :title => t('layouts.help_title')) %></li>