Remove inline javascript from page header

[rails.git] / app / views / layouts / _header.html.erb

diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
index c4f2ebad6d563e04b9397eff14e85d627fad5056..a8811c6c9c39ff27842ba803faef9aa651fb5036 100644 (file)
--- a/app/views/layouts/_header.html.erb
+++ b/app/views/layouts/_header.html.erb
@@ -1,56 +1,72 @@
-<header>
+<header class="closed">

   <h1>
   <h1>

-    <a href="<% root_path %>" class="geolink llz layers">
-      <%= image_tag "osm_logo.png", :alt => t('layouts.logo.alt_text'), :class => 'logo' %>
+    <a href="<%= root_path %>" class="geolink">
+      <picture>
+        <source srcset="<%= image_path "osm_logo.svg" %>" type="image/svg+xml"></source>
+        <%= image_tag "osm_logo.png", :srcset => image_path("osm_logo.svg"), :alt => t('layouts.logo.alt_text'), :class => 'logo' %>
+      </picture>

       <%= t 'layouts.project_name.h1' %>
     </a>
   </h1>
       <%= t 'layouts.project_name.h1' %>
     </a>
   </h1>

+  <a href="#" id="menu-icon"></a>

   <nav class='primary'>
   <nav class='primary'>

+    <%= content_for :header %>

     <ul>
     <ul>

-      <li id="view_tab">
-        <%= link_to t('layouts.view'), root_path, :class => 'tab geolink llz layers' %>
-      </li><li id="edit_tab" class="dropdown">
-      <%= link_to t('layouts.edit'), edit_path, :class => "tab geolink llz object",
-                  :id => 'editanchor',
-                  :data => { :editor => preferred_editor }
-      %><a class='dropdown-toggle' data-toggle='dropdown' href='#'><b class="caret"></b></a>
-      <ul class='dropdown-menu'>
-        <% Editors::RECOMMENDED_EDITORS.each do |editor| %>
-          <li>
-            <%= link_to t('layouts.edit_with', :editor => t("editor.#{editor}.description")),
-                        edit_path(:editor => editor),
-                        :data => { :editor => editor },
-                        :class => "geolink llz object" %>
-          </li>
-        <% end %>
-      </ul>
-      </li><li id="history_tab">
-        <%= link_to t('layouts.history'), browse_changesets_path, :class => 'tab geolink llz' %>
-      </li><li id="export_tab">
-        <%= link_to t('layouts.export'), export_path, :class => 'tab geolink llz' %>
+      <li id="edit_tab" class="dropdown <%= current_page_class(edit_path) %>">
+        <%= link_to t('layouts.edit'), edit_path, :class => "tab geolink editlink",
+                    :id => 'editanchor',
+                    :data => { :editor => preferred_editor }
+        %><a class='dropdown-toggle' data-toggle='dropdown' href='#'><b class="caret"></b></a>
+        <ul class='dropdown-menu'>
+          <% Editors::RECOMMENDED_EDITORS.each do |editor| %>
+            <li>
+              <%= link_to t('layouts.edit_with', :editor => t("editor.#{editor}.description")),
+                          edit_path(:editor => editor),
+                          :data => { :editor => editor },
+                          :class => "geolink editlink" %>
+            </li>
+          <% end %>
+        </ul>
+      </li>
+      <li id="history_tab" class="<%= current_page_class(history_path) %>">
+        <%= link_to t('layouts.history'), history_path, :class => 'tab geolink' %>
+      </li>
+      <li id="export_tab" class="<%= current_page_class(export_path) %>">
+        <%= link_to t('layouts.export'), export_path, :class => 'tab geolink' %>

       </li>
     </ul>
   </nav>
   <nav class='secondary'>
     <ul>
       </li>
     </ul>
   </nav>
   <nav class='secondary'>
     <ul>

-      <li><%= link_to t('layouts.gps_traces'), traces_path %></li>
-      <li><%= link_to t('layouts.user_diaries'), diary_path %></li>
-      <li><%= link_to t('layouts.help'), help_path %></li>
-      <li><%= link_to t('layouts.about'), about_path %></li>
+      <li class="compact-hide <%= current_page_class(traces_path) %>"><%= link_to t('layouts.gps_traces'), traces_path %></li>
+      <li class="compact-hide <%= current_page_class(diary_path) %>"><%= link_to t('layouts.user_diaries'), diary_path %></li>
+      <li class="compact-hide <%= current_page_class(copyright_path) %>"><%= link_to t('layouts.copyright'), copyright_path %></li>
+      <li class="compact-hide <%= current_page_class(help_path) %>"><%= link_to t('layouts.help'), help_path %></li>
+      <li class="compact-hide <%= current_page_class(about_path) %>"><%= link_to t('layouts.about'), about_path %></li>
+      <li id="compact-secondary-nav" class="dropdown">
+        <a class="dropdown-toggle" data-toggle="dropdown" href="#"><%= t 'layouts.more' %> <b class="caret"></b></a>
+        <ul class="dropdown-menu">
+          <li class="<%= current_page_class(traces_path) %>"><%= link_to t('layouts.gps_traces'), traces_path %></li>
+          <li class="<%= current_page_class(diary_path) %>"><%= link_to t('layouts.user_diaries'), diary_path %></li>
+          <li class="<%= current_page_class(copyright_path) %>"><%= link_to t('layouts.copyright'), copyright_path %></li>
+          <li class="<%= current_page_class(help_path) %>"><%= link_to t('layouts.help'), help_path %></li>
+          <li class="<%= current_page_class(about_path) %>"><%= link_to t('layouts.about'), about_path %></li>
+        </ul>
+      </li>

     </ul>
     </ul>

-    <% if @user %>
-      <div class='dropdown user-menu'>
+    <% if @user && @user.id %>
+      <div class='dropdown user-menu logged-in'>

         <a class='dropdown-toggle' data-toggle='dropdown' href="#">
         <a class='dropdown-toggle' data-toggle='dropdown' href="#">

-          <%= user_thumbnail_tiny(@user, :size => 20, :width => 20, :height => 20)
+          <%= user_thumbnail_tiny(@user, :size => 25, :width => 25, :height => 25)

           %><%= render :partial => 'layouts/inbox'
           %><%= render :partial => 'layouts/inbox'

-        %><span class='username'><%= @user.display_name %></span>
-          <b class="caret"></b>
+        %><span class="user-button"><span class='username'><%= @user.display_name %></span>
+          <b class="caret"></b></span>

         </a>
         <ul class='dropdown-menu'>
           <li>
             <%= link_to inbox_path(:display_name => @user.display_name) do %>
               <span class='count-number'><%= number_with_delimiter(@user.new_messages.size) %></span>
         </a>
         <ul class='dropdown-menu'>
           <li>
             <%= link_to inbox_path(:display_name => @user.display_name) do %>
               <span class='count-number'><%= number_with_delimiter(@user.new_messages.size) %></span>

-              <%= t('message.inbox.my_inbox') %>
+              <%= t('user.view.my messages') %>

             <% end %>
           </li>
           <li>
             <% end %>
           </li>
           <li>


@@ -69,7 +85,7 @@
         </ul>
       </div>
     <% else %>
         </ul>
       </div>
     <% else %>

-      <ul class="user-menu">
+      <ul class="user-menu clearfix">

         <li><%= link_to t('layouts.log_in'), login_path(:referer => request.fullpath) %></li>
         <li><%= link_to t('layouts.sign_up'), user_new_path %></li>
       </ul>
         <li><%= link_to t('layouts.log_in'), login_path(:referer => request.fullpath) %></li>
         <li><%= link_to t('layouts.sign_up'), user_new_path %></li>
       </ul>