* Make RecentChanges use a table and some CSS, should be much more readable.

No code changes involved.
* Encode & in diffurl in examples, to conform to pedantic rules.

IkiWiki/Rcs/svn.pm

@@ -126,7 +126,6 @@ sub rcs_recentchanges ($) { #{{{
 		foreach my $msgline (split(/\n/, $logentry->{msg})) {
 			push @message, { line => escapeHTML($msgline) };
 		}
-		unshift @message, { line => "\n" } if @message > 1;
 
 		my $committype="web";
 		if (defined $message[0] &&

basewiki/style.css

@@ -30,6 +30,15 @@
 	width: 100%;
 }      
 
+/* Stuff for the RecentChanges table. */
+.changeheader {
+	background: #eee;
+	color: black !important;
+}
+.changetime {
+	white-space: nowrap;
+}
+
 /* Used for adding a blog page. */
 #blogform {
 	padding: 10px 10px;

debian/changelog

@@ -42,8 +42,11 @@ ikiwiki (1.8) UNRELEASED; urgency=low
     is a savings of 2 pointless fork/execs per link calculation, which
     results in ~25% speedup of ikiwiki building its own doc wiki, and
     about 35% speedup displaying RecentChanges!
+  * Make RecentChanges use a table and some CSS, should be much more readable.
+    No code changes involved.
+  * Encode & in diffurl in examples, to conform to pedantic rules. 
 
- -- Joey Hess <joeyh@debian.org>  Mon,  3 Jul 2006 23:33:57 -0400
+ -- Joey Hess <joeyh@debian.org>  Tue,  4 Jul 2006 01:42:28 -0400
 
 ikiwiki (1.7) unstable; urgency=low
 

doc/ikiwiki.setup

@@ -21,7 +21,7 @@ use IkiWiki::Setup::Standard {
 	# Subversion stuff.
 	rcs => "svn",
 	#historyurl => "http://svn.myhost/trunk/[[file]]",
-	#diffurl => "http://svn.someurl/trunk/[[file]]?root=wiki&r1=[[r1]]&r2=[[r2]]",
+	#diffurl => "http://svn.someurl/trunk/[[file]]?root=wiki&r1=[[r1]]&r2=[[r2]]",
 	svnrepo => "/svn/wiki",
 	svnpath => "trunk",
 

doc/todo/html.mdwn

@@ -1,5 +1,4 @@
-Create some nice stylesheets. Make RecentChanges use table for
-formatting, and images to indicate web vs svn commits and to link to diffs.
+Create some nice(r) stylesheets.
 
-All of this should be doable w/o touching a single line of code, just
-editing the [[templates]] and/or editing [[style.css]] BTW.
+Should be doable w/o touching a single line of code, just
+editing the [[templates]] and/or editing [[style.css]].

doc/usage.mdwn

@@ -158,7 +158,7 @@ These options configure the wiki.
 
   Specifies the email address that ikiwiki should use for sending email.
 
-* --diffurl http://url/trunk/\[[file]]?root=wiki&r1=\[[r1]]&r2=\[[r2]]
+* --diffurl http://url/trunk/\[[file]]?root=wiki&r1=\[[r1]]&r2=\[[r2]]
 
   Specifies the url to link to for a diff of changes to a page. In the url,
   "\[[file]]" is replaced with the file to browse, "\[[r1]]" is the old

templates/recentchanges.tmpl

@@ -18,28 +18,49 @@
 </div>
 
 <div id="content">
-<ul>
+<br />
+<table border="1" frame="border" rules="groups">
+<thead>
+	<tr class="changeheader">
+		<th align="left">user</th>
+		<th align="left">time</th>
+		<th align="left">changes</th>
+	</tr>
+</thead>
 <TMPL_LOOP NAME="CHANGELOG">
-	<li> <!-- <TMPL_VAR NAME="REV"> -->
+<tbody>
+	<!-- <TMPL_VAR NAME="REV"> -->
+	<!-- <TMPL_VAR NAME="COMMITTYPE"> -->
+	<tr class="changeheader">
+		<td><TMPL_VAR NAME="USER"></td>
+		<td class="changetime"><TMPL_VAR NAME="WHEN"></td>
+		<td>
 		<TMPL_LOOP NAME="PAGES">
-			<TMPL_VAR NAME="LINK">
 			<TMPL_IF NAME="DIFFURL">
-				(<a href="<TMPL_VAR NAME="DIFFURL">">diff</a>)
+				<a href="<TMPL_VAR NAME="DIFFURL">">
+				<img alt="diff" src="wikiicons/diff.png" style="border-style: none" />
+				</a>
+				<TMPL_VAR NAME="LINK">
+			<TMPL_ELSE>
+				<TMPL_VAR NAME="LINK">
 			</TMPL_IF>
 		</TMPL_LOOP>
-		<br />
-		<!-- <TMPL_VAR NAME="COMMITTYPE"> -->
-		changed <TMPL_VAR NAME="WHEN"> by <TMPL_VAR NAME="USER">:
-		<i>
-		<TMPL_LOOP NAME="MESSAGE">
-			<TMPL_IF NAME="LINE">
-				<TMPL_VAR NAME="LINE"><br />
-			</TMPL_IF>
-		</TMPL_LOOP>
-		</i>
-	</li>
+		</td>
+	</tr>
+	<tr>
+		<td colspan="3">
+			<i>
+			<TMPL_LOOP NAME="MESSAGE">
+				<TMPL_IF NAME="LINE">
+					<TMPL_VAR NAME="LINE"><br />
+				</TMPL_IF>
+			</TMPL_LOOP>
+			</i>
+		</td>
+	</tr>
+</tbody>
 </TMPL_LOOP>
-</ul>
+</table>
 </div>
 
 <!-- from <TMPL_VAR NAME=WIKINAME> -->