+<% this_colour = cycle('lightgrey', 'white') # can only call once for some dumb reason %>
+
<tr class="inbox-row<%= "-unread" if not message_summary.message_read? %>">
- <td class="inbox-sender"><%= link_to message_summary.sender.display_name , :controller => 'user', :action => message_summary.sender.display_name %></td>
- <td class="inbox-subject"><%= link_to message_summary.title , :controller => 'message', :action => 'read', :message_id => message_summary.id %></td>
- <td class="inbox-sent"><%= message_summary.sent_on %></td>
+ <td class="inbox-sender" bgcolor='<%= this_colour %>'><%= link_to message_summary.sender.display_name , :controller => 'user', :action => message_summary.sender.display_name %></td>
+ <td class="inbox-subject" bgcolor='<%= this_colour %>'><%= link_to message_summary.title , :controller => 'message', :action => 'read', :message_id => message_summary.id %></td>
+ <td class="inbox-sent" bgcolor='<%= this_colour %>'><%= message_summary.sent_on %></td>
<% if message_summary.message_read? %>
<td><%= button_to 'Mark as unread', :controller => 'message', :action => 'mark', :message_id => message_summary.id, :mark => 'unread' %></td>
<% else %>
<table>
<tr>
- <th>From</th>
+ <th align="right">From</th>
<td><%= link_to @message.sender.display_name, :controller => 'user', :action => 'view', :display_name => @message.sender.display_name %></td>
</tr>
<tr>
- <th>Subject</th>
+ <th align="right">Subject</th>
<td><%= @message.title %></td>
</tr>
<tr>
- <th>Date</th>
+ <th align="right">Date</th>
<td><%= @message.sent_on %></td>
</tr>
<tr>
- <th>Message</th>
+ <th></th>
<td><%= @message.body %></td>
</tr>
</table>