From: Steve Coast <steve@asklater.com>
Date: Tue, 4 Sep 2007 14:20:42 +0000 (+0000)
Subject: a few message prettyness things
X-Git-Tag: live~8183
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/2c0cd2730c369410a677bcd30e6233534e7fc5ad

a few message prettyness things
---

diff --git a/app/views/message/_message_summary.rhtml b/app/views/message/_message_summary.rhtml
index 56f6bb132..02972728e 100644
--- a/app/views/message/_message_summary.rhtml
+++ b/app/views/message/_message_summary.rhtml
@@ -1,7 +1,9 @@
+<% 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 %>
diff --git a/app/views/message/read.rhtml b/app/views/message/read.rhtml
index b84fea4c7..7a0d2bbe1 100644
--- a/app/views/message/read.rhtml
+++ b/app/views/message/read.rhtml
@@ -2,19 +2,19 @@
 
 <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>