Fix for #1076

New feature to allow users to collapse and expand long revision
messages on the history page.

Author: vbro

web/default/style.css

@@ -493,6 +493,10 @@ table#revisions tbody tr td p {
 	margin: 1ex 0;
 }
 
+.rev-message-hidden {
+  display: none;
+}
+
 
 /* *** diff page *** */
 #diffbar { /* diff navbar: contains the tabs to select diff format */

web/history.jsp

@@ -279,8 +279,10 @@ revision2 = revision2 >= hist.getHistoryEntries().size() ? hist.getHistoryEntrie
                 %><%= author %><%
                 }
                 %></td>
-            <td><a name="<%= rev %>"></a><p><%
+            <td><a name="<%= rev %>"></a><%
+                int summaryLength = 200;
                 String cout = Util.htmlize(entry.getMessage());
+
                 if (bugPage != null && bugPage.length() > 0) {
                     cout = bugPattern.matcher(cout).replaceAll("<a href=\""
                         + bugPage + "$1\">$1</a>");
@@ -289,7 +291,26 @@ revision2 = revision2 >= hist.getHistoryEntries().size() ? hist.getHistoryEntrie
                     cout = reviewPattern.matcher(cout).replaceAll("<a href=\""
                         + reviewPage + "$1\">$1</a>");
                 }
-                %><%= cout %></p><%
+                
+                boolean showSummary = false;
+                String coutSummary = entry.getMessage();
+                if (coutSummary.length() > summaryLength) {
+                    showSummary = true;
+                    coutSummary = coutSummary.substring(0, summaryLength - 1);
+                    coutSummary = Util.htmlize(coutSummary);
+                }
+
+                if (showSummary) {
+                    %>
+                    <p class="rev-message-summary"><%= coutSummary %></p>
+                    <p class="rev-message-full rev-message-hidden"><%= cout %></p>
+                    <p class="rev-message-toggle" data-toggle-state="less"><a class="rev-toggle-a" href="#">show more ... </a></p>
+                    <%
+                }
+                else {
+                     %><p class="rev-message-full"><%= cout %></p><%
+                }
+
                 Set<String> files = entry.getFiles();
                 if (files != null) {
                 %><div class="filelist-hidden"><br/><%

web/utils.js

@@ -854,6 +854,28 @@ function togglediffs() {
     );
 }
 
+$(document).ready(function() {
+  $(".rev-toggle-a").click(function() {
+    var toggleState = $(this).closest("p").attr("data-toggle-state");
+    var thisCell = $(this).closest("td");
+
+    if (toggleState == "less") {
+      $(this).closest("p").attr("data-toggle-state", "more");
+      thisCell.find(".rev-message-summary").addClass("rev-message-hidden");
+      thisCell.find(".rev-message-full").removeClass("rev-message-hidden");
+      $(this).html("... show less");
+    }
+    else if (toggleState == "more") {
+      $(this).closest("p").attr("data-toggle-state", "less");
+      thisCell.find(".rev-message-full").addClass("rev-message-hidden");
+      thisCell.find(".rev-message-summary").removeClass("rev-message-hidden");
+      $(this).html("show more ...");
+    }
+
+    return false;
+  });
+});
+
 function selectAllProjects() {
     $("#project *").attr("selected", "selected");
 }