changeset 141:c29197bd2907

fixed listings, of commits, added slugs for longer commits messages
author Marcin Kuzminski <marcin@python-works.com>
date Sun, 09 May 2010 16:51:42 +0200
parents b5e59e2b5cfe
children f7218849798a
files pylons_app/templates/shortlog/shortlog_data.html pylons_app/templates/summary.html
diffstat 2 files changed, 36 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/pylons_app/templates/shortlog/shortlog_data.html	Sun May 09 15:06:00 2010 +0200
+++ b/pylons_app/templates/shortlog/shortlog_data.html	Sun May 09 16:51:42 2010 +0200
@@ -3,17 +3,31 @@
 from pylons_app.lib import filters
 %>
 <table>
+<%def name="message_slug(msg)">
+	<%
+	limit = 60
+	if len(msg) > limit:
+		return msg[:limit]+'...'
+	else:
+		return msg
+	%>
+</%def>
 %for cnt,cs in enumerate(c.repo_changesets):
 	<tr class="parity${cnt%2}">
 		<td>${cs._ctx.date()|n,filters.age}</td>
 		<td title="${cs.author}">${cs.author|n,filters.person}</td>
-		<td>${h.link_to(cs.message,h.url('changeset_home',repo_name=c.repo_name,revision=cs._short))}
-			<span class="logtags">
-				<span class="branchtag">${cs.branch}</span>
-				%for tag in cs.tags:
-					<span class="tagtag">${tag}</span>
-				%endfor
-			</span>		
+		<td>
+			${h.link_to(message_slug(cs.message),
+			h.url('changeset_home',repo_name=c.repo_name,revision=cs._short),
+			title=cs.message)}
+		</td>
+		<td>
+				<span class="logtags">
+					<span class="branchtag">${cs.branch}</span>
+					%for tag in cs.tags:
+						<span class="tagtag">${tag}</span>
+					%endfor
+				</span>
 		</td>
 		<td class="nowrap">
 		${h.link_to(_('changeset'),h.url('changeset_home',repo_name=c.repo_name,revision=cs._short))}
--- a/pylons_app/templates/summary.html	Sun May 09 15:06:00 2010 +0200
+++ b/pylons_app/templates/summary.html	Sun May 09 16:51:42 2010 +0200
@@ -49,13 +49,25 @@
 
     <h2>${h.link_to(_('Changes'),h.url('changelog_home',repo_name=c.repo_name))}</h2>
     <table>
+	<%def name="message_slug(msg)">
+		<%
+		limit = 60
+		if len(msg) > limit:
+			return msg[:limit]+'...'
+		else:
+			return msg
+		%>
+	</%def>    
 	%for cnt,cs in enumerate(c.repo_changesets):
 		<tr class="parity${cnt%2}">
 			<td>${cs._ctx.date()|n,filters.age}</td>
-			<td>${cs.author}</td>
+			<td>${cs.author|n,filters.person}</td>
 			<td>
-				
-				${h.link_to(cs.message,h.url('changeset_home',repo_name=c.repo_name,revision=cs._short))}
+				${h.link_to(message_slug(cs.message),
+				h.url('changeset_home',repo_name=c.repo_name,revision=cs._short),
+				title=cs.message)}
+			</td>
+			<td>
 				<span class="logtags">
 					<span class="branchtag">${cs.branch}</span>
 					%for tag in cs.tags: