Merge pull request #67 from finnlabs/fix/18330-harmonize-headings

[18330] Use toolbar

app/views/meeting_contents/diff.html.erb

@@ -19,15 +19,16 @@ See doc/COPYRIGHT.md for more details.
 
 ++#%>
 
-<% content_for :action_menu_specific do %>
-  <%= link_to_if_authorized(l(:label_history), {:controller => '/' + @content_type.pluralize, :action => 'history', :meeting_id => @meeting}, :class => 'icon icon-wiki') %>
-<% end %>
-
 <% html_title "#{l(:label_meeting_diff)}: #{@meeting.title}" %>
-
-<h2><%= l(:"label_#{@content_type}") %>: <%= link_to @meeting, @meeting %></h2>
-
-<%= render :partial => 'layouts/action_menu_specific' %>
+<%= toolbar title: "#{l(:"label_#{@content_type}")}: #{link_to @meeting, @meeting}" do %>
+  <% if authorize_for(@content_type.pluralize, :history) %>
+    <li class="toolbar-item">
+      <%= link_to({controller: "/#{@content_type.pluralize}", action: 'history', meeting_id: @meeting }, class: 'button') do %>
+        <i class="button--icon icon-wiki"></i> <%= l(:label_history) %>
+      <% end %>
+    </li>
+  <% end %>
+<% end %>
 
 <p>
 <%= l(:label_version) %> <%= link_to @diff.content_from.version, send(:"#{@content_type}_version_path", @meeting, @diff.content_from.version) %>

app/views/meeting_contents/history.html.erb

@@ -21,7 +21,7 @@ See doc/COPYRIGHT.md for more details.
 
 <% html_title "#{l(:label_history)}: #{@meeting.title}" %>
 
-<h2><%= l(:"label_#{@content_type}") %>: <%= link_to @meeting, @meeting %></h2>
+<%= toolbar title: "#{l(:"label_#{@content_type}")}: #{link_to @meeting, @meeting}" %>
 
 <h3><%= l(:label_history) %></h3>
 

app/views/meetings/edit.html.erb

@@ -21,7 +21,7 @@ See doc/COPYRIGHT.md for more details.
 
 <% html_title "#{l(:label_meeting_edit)}: #{@meeting.title}" %>
 
-<h2><%= l(:label_meeting) %> #<%= @meeting.id %></h2>
+<%= toolbar title: "#{l(:label_meeting)} ##{@meeting.id}" %>
 <%= labelled_tabular_form_for @meeting, :url => {:controller => '/meetings', :action => 'update'}, :html => {:id => 'meeting-form', :method => :put} do |f| -%>
   <%= render :partial => 'form', :locals => {:f => f} %>
 <%= submit_tag l(:button_save), class: 'button -highlight' %>

app/views/meetings/index.html.erb

@@ -21,8 +21,7 @@ See doc/COPYRIGHT.md for more details.
 
 <% html_title l(:label_meeting_plural) %>
 
-<h2><%= l(:label_meeting_plural)%></h2>
-
+<%= toolbar title: l(:label_meeting_plural) %>
 <% if @meetings_by_start_year_month_date.empty? -%>
 <p class="nodata"><%= l(:label_no_data) %></p>
 <% else -%>

app/views/meetings/new.html.erb

@@ -20,8 +20,7 @@ See doc/COPYRIGHT.md for more details.
 ++#%>
 
 <% html_title l(:label_meeting_new) %>
-
-<h2><%= l(:label_meeting_new) %></h2>
+<%= toolbar title: l(:label_meeting_new) %>
 <%= labelled_tabular_form_for @meeting, :url => {:controller => '/meetings', :action => 'create', :project_id => @project}, :html => {:id => 'meeting-form'} do |f| -%>
   <%= render :partial => 'form', :locals => {:f => f} %>
   <%= submit_tag l(:button_create), class: 'button -highlight' %>

app/views/meetings/show.html.erb

@@ -19,18 +19,40 @@ See doc/COPYRIGHT.md for more details.
 
 ++#%>
 
-<% content_for :action_menu_specific do %>
-  <%= watcher_link @meeting, User.current %>
-  <%= link_to_if_authorized(l(:button_edit), {:controller => '/meetings', :action => 'edit', :id => @meeting}, :class => 'icon icon-edit', :accesskey => accesskey(:edit))%>
-  <%= link_to_if_authorized(l(:button_copy), {:controller => '/meetings', :action => 'copy', :id => @meeting}, :class => 'icon icon-copy')%>
-  <%= link_to_if_authorized(l(:button_delete), {:controller => '/meetings', :action => 'destroy', :id => @meeting}, :method => :delete, :confirm => l(:text_are_you_sure), :class => 'icon icon-delete')%>
-<% end %>
-
 <% html_title "#{l(:label_meeting)}: #{@meeting.title}" %>
+<%= toolbar title: "#{l(:label_meeting)}: #{link_to @meeting}" do %>
+  <% unless User.current.anonymous? %>
+    <li class="toolbar-item">
+      <div class="button">
+        <%= watcher_link @meeting, User.current %>
+      </div>
+    </li>
+  <% end %>
+  <% if authorize_for(:meetings, :edit) %>
+    <li class="toolbar-item">
+      <%= link_to({:controller => '/meetings', :action => 'edit', :id => @meeting}, class: 'button',:accesskey => accesskey(:edit)) do%>
+        <i class="button--icon icon-edit"></i> <%= l(:button_edit) %>
+      <% end %>
+    </li>
+  <% end %>
+  <% if authorize_for(:meetings, :copy) %>
+    <li class="toolbar-item">
+      <%= link_to({:controller => '/meetings', :action => 'copy', :id => @meeting}, class: 'button') do %>
+        <i class="button--icon icon-copy"></i> <%= l(:button_copy) %>
+      <% end %>
+    </li>
+  <% end %>
+  <% if authorize_for(:meetings, :delete) %>
+    <li class="toolbar-item">
+      <%= link_to({:controller => '/meetings', :action => 'destroy', :id => @meeting}, class: 'button') do %>
+        <i class="button--icon icon-delete"></i> <%= l(:button_delete) %>
+      <% end %>
+    </li>
+  <% end %>
+<% end %>
 
 <div class="top-page">
   <%= render :partial => 'layouts/action_menu_specific' %>
-  <h2 title="<%= h @meeting.title %>"><%= l(:label_meeting) %>: <%= link_to @meeting %></h2>
 </div>
 
 <div class="meeting details box">