# HG changeset patch
# User olivier
# Date 1420626361 -3600
#      Wed Jan 07 11:26:01 2015 +0100
# Node ID 3069fb9e37e6c67801001d84d52c6822d98c03ed
# Parent  6433fb0650326cd73296f7fe9a1b1f135bbea320
#42140: add code to make History and Description collapsable/expandable in the same way as grouped issue lists

diff -r 6433fb065032 -r 3069fb9e37e6 app/views/issues/_history.html.erb
--- a/app/views/issues/_history.html.erb	Fri Dec 12 09:20:05 2014 +0100
+++ b/app/views/issues/_history.html.erb	Wed Jan 07 11:26:01 2015 +0100
@@ -1,12 +1,22 @@
 <% reply_links = authorize_for('issues', 'edit') -%>
+<table width="100%">
+  <tbody>
 <% for journal in journals %>
-  <div id="change-<%= journal.id %>" class="<%= journal.css_classes %>">
+    <tr class="group open"><td>
+    <div id="change-<%= journal.id %>" class="<%= journal.css_classes %>">
     <div id="note-<%= journal.indice %>">
     <h4><%= link_to "##{journal.indice}", {:anchor => "note-#{journal.indice}"}, :class => "journal-link" %>
     <%= avatar(journal.user, :size => "24") %>
+    
+    <span class="expander" onclick="toggleRowGroup(this);">&nbsp;</span>
     <%= authoring journal.created_on, journal.user, :label => :label_updated_time_by %>
+    <%= link_to_function("#{l(:button_collapse_all)}/#{l(:button_expand_all)}",
+                         "toggleAllRowGroups(this)", :class => 'toggle-all') %> 
+
     <%= content_tag('span', l(:field_is_private), :class => 'private') if journal.private_notes? %></h4>
 
+    </td></tr>
+    <tr><td>
     <% if journal.details.any? %>
     <ul class="details">
       <% details_to_strings(journal.visible_details).each do |string| %>
@@ -18,6 +28,9 @@
     </div>
   </div>
   <%= call_hook(:view_issues_history_journal_bottom, { :journal => journal }) %>
+  </td></tr>
 <% end %>
+  </tbody>
+</table>
 
 <% heads_for_wiki_formatter if User.current.allowed_to?(:edit_issue_notes, issue.project) || User.current.allowed_to?(:edit_own_issue_notes, issue.project) %>
diff -r 6433fb065032 -r 3069fb9e37e6 app/views/issues/show.html.erb
--- a/app/views/issues/show.html.erb	Fri Dec 12 09:20:05 2014 +0100
+++ b/app/views/issues/show.html.erb	Wed Jan 07 11:26:01 2015 +0100
@@ -71,16 +71,27 @@
 <% if @issue.description? || @issue.attachments.any? -%>
 <hr />
 <% if @issue.description? %>
-<div class="description">
-  <div class="contextual">
-  <%= link_to l(:button_quote), quoted_issue_path(@issue), :remote => true, :method => 'post', :class => 'icon icon-comment' if authorize_for('issues', 'edit') %>
+<table width="100%">
+  <tbody>
+    <tr class="group open"><td>
+    <div class="description">
+    <div class="contextual">
+    <%= link_to l(:button_quote), quoted_issue_path(@issue), :remote => true, :method => 'post', :class => 'icon icon-comment' if authorize_for('issues', 'edit') %>
+    </div>
+  
+    <p>
+      <span class="expander" onclick="toggleRowGroup(this);">&nbsp;</span>
+      <strong><%=l(:field_description)%></strong>
+    </p>
+  </td></tr>
+  <tr><td>
+    <div class="wiki">
+    <%= textilizable @issue, :description, :attachments => @issue.attachments %>
+    </div>
+    </td></tr>
   </div>
-
-  <p><strong><%=l(:field_description)%></strong></p>
-  <div class="wiki">
-  <%= textilizable @issue, :description, :attachments => @issue.attachments %>
-  </div>
-</div>
+  </tbody>
+</table> 
 <% end %>
 <%= link_to_attachments @issue, :thumbnails => true %>
 <% end -%>