Merge pull request #714 from vladak/git-hist-align

left-align comments even for history view of directories

Author: Vladimir Kotal

web/default/style.css

@@ -287,10 +287,6 @@ table#revisions {
 	margin-left: 2em;
 }
 
-#revisions .filelist { /* "Show modified files" */
-	display: inline;
-}
-
 #revisions .filelist-hidden { /* "Hide modified files" */
 	display: none;
 }
@@ -307,20 +303,28 @@ table#revisions {
 	display: none;
 }
 
-#revisions tbody td:first-child {
+#revisions td {
+	padding: 0.5ex;
+	text-align: center;
+}
+
+#revisions td:first-child {
 	/* rev number - git/mercurial revs numbers look bad with prop. fonts */
 	font-family: monospace;
 	text-align: right;
 }
 
-#revisions tbody td:nth-child(2), #revisions tbody td:nth-child(3) {
-	/* Compare and Date column */
-	text-align: center;
+#revisions td > p {
+	/* log message in comment column */
+	margin: 0;
+	text-align: left;
 }
 
-#revisions tbody td:nth-child(4) { /* author column */
-	padding: 0 1ex;
-	text-align: center;
+#revisions td > div {
+	/* modified file list in comment column */
+	font-family: monospace;
+	text-align: left;
+	float: left;
 }
 
 #revisions tbody td.revtags { /* tags row */

web/history.jsp

@@ -105,10 +105,10 @@ document.domReady.push(function() {domReadyHistory();});
             <th>Comments <%
             if (hist.hasFileList()) {
                 %><a href="#" onclick="javascript: toggle_filelist(); return false;">
-                    <span class="filelist-hidden">
-                    (&lt;&lt;&lt; Hide modified files)</span>
-                    <span class="filelist">
-                    (Show modified files &gt;&gt;&gt;)</span></a><%
+                    <div class="filelist-hidden">
+                    (&lt;&lt;&lt; Hide modified files)</div>
+                    <div class="filelist">
+                    (Show modified files &gt;&gt;&gt;)</div></a><%
             }
             %>
             </th>
@@ -189,7 +189,7 @@ document.domReady.push(function() {domReadyHistory();});
                 %><%= author %><%
                 }
                 %></td>
-            <td><%
+            <td><p><%
                 String cout = Util.htmlize(entry.getMessage());
                 if (bugPage != null && bugPage.length() > 0) {
                     cout = bugPattern.matcher(cout).replaceAll("<a href=\""
@@ -199,10 +199,10 @@ document.domReady.push(function() {domReadyHistory();});
                     cout = reviewPattern.matcher(cout).replaceAll("<a href=\""
                         + reviewPage + "$1\">$1</a>");
                 }
-                %><%= cout %><%
+                %><%= cout %></p><%
                 Set<String> files = entry.getFiles();
                 if (files != null) {
-                %><span class="filelist-hidden"><br/><%
+                %><div class="filelist-hidden"><br/><%
                     for (String ifile : files) {
                         String jfile = Util.stripPathPrefix(path, ifile);
                         if (rev == "") {
@@ -213,7 +213,7 @@ document.domReady.push(function() {domReadyHistory();});
 <a class="h" href="<%= context + Prefix.XREF_P + ifile %>?r=<%= rev %>"><%= jfile %></a><br/><%
                         }
                     }
-                %></span><%
+                %></div><%
                 }
                 %></td>
         </tr><%

web/offwhite/style.css

@@ -318,10 +318,6 @@ table#revisions {
 	margin-left: 2em;
 }
 
-#revisions .filelist { /* "Show modified files" */
-	display: inline;
-}
-
 #revisions .filelist-hidden { /* "Hide modified files" */
 	display: none;
 }
@@ -338,20 +334,28 @@ table#revisions {
 	display: none;
 }
 
+#revisions td {
+	padding: 0.5ex;
+	text-align: center;
+}
+
 #revisions tbody td:first-child {
 	/* rev number - git/mercurial revs numbers look bad with prop. fonts */
 	font-family: monospace;
 	text-align: right;
 }
 
-#revisions tbody td:nth-child(2), #revisions tbody td:nth-child(3) {
-	/* Compare and Date column */
-	text-align: center;
+#revisions td > p {
+	/* log message in comment column */
+	margin: 0;
+	text-align: left;
 }
 
-#revisions tbody td:nth-child(4) { /* author column */
-	padding: 0 1ex;
-	text-align: center;
+#revisions td > div {
+	/* modified file list in comment column */
+	font-family: monospace;
+	text-align: left;
+	float: left;
 }
 
 #revisions tbody td.revtags { /* tags row */

web/polished/style.css

@@ -371,7 +371,6 @@ table#revisions {
 }
 
 #revisions .filelist { /* "Show modified files" */
-	display: inline;
 	color: #ffdd80;
 }
 
@@ -391,20 +390,28 @@ table#revisions {
 	display: none;
 }
 
+#revisions td {
+	padding: 0.5ex;
+	text-align: center;
+}
+
 #revisions tbody td:first-child {
 	/* rev number - git/mercurial revs numbers look bad with prop. fonts */
 	font-family: monospace;
 	text-align: right;
 }
 
-#revisions tbody td:nth-child(2), #revisions tbody td:nth-child(3) {
-	/* Compare and Date column */
-	text-align: center;
+#revisions td > p {
+	/* log message in comment column */
+	margin: 0;
+	text-align: left;
 }
 
-#revisions tbody td:nth-child(4) { /* author column */
-	padding: 0 1ex;
-	text-align: center;
+#revisions td > div {
+	/* modified file list in comment column */
+	font-family: monospace;
+	text-align: left;
+	float: left;
 }
 
 #revisions tbody td.revtags { /* tags row */

web/utils.js

@@ -325,7 +325,7 @@ function add_highlight() {
 }
 
 function toggle_filelist() {
-    $("span").each(
+    $("div").each(
         function() {
             if (this.className == "filelist") {
                 this.setAttribute("style", "display: none;");