Use user_path links. Fixes #1785

[rails.git] / app / views / message / read.html.erb

diff --git a/app/views/message/read.html.erb b/app/views/message/read.html.erb
index 8a7b6b1563c235340f6969e42a320f5b53b3497e..c23eabe95acf59d99f7028c617f6e75b563b39aa 100644 (file)
--- a/app/views/message/read.html.erb
+++ b/app/views/message/read.html.erb
@@ -1,70 +1,40 @@
-<% if @user == @message.recipient %>
-
-<h2><%= t'message.read.reading_your_messages' %></h2>
-
-<table>
-  <tr>
-    <th align="right"><%= t'message.read.from' %></th>
-    <td>
-     <% if @message.sender.image %>
-        <%= image_tag url_for_file_column(@message.sender, "image") %>
-      <% end %>
-  
-<%= link_to h(@message.sender.display_name), :controller => 'user', :action => 'view', :display_name => @message.sender.display_name %></td>
-  </tr>
-  <tr>
-    <th align="right"><%= t'message.read.subject' %></th>
-    <td><%= h(@message.title) %></td>
-  </tr>
-  <tr>
-    <th align="right"><%= t'message.read.date' %></th>
-    <td><%= l @message.sent_on %></td>
-  </tr>
-  <tr>
-    <th></th>
-    <td><%= htmlize(@message.body) %></td>
-  </tr>
-</table>
-
-<br />
-
-<table>
-  <tr>
-    <td><%= button_to t('message.read.reply_button'), :controller => 'message', :action => 'reply', :message_id => @message.id %></td>
-    <td><%= button_to t('message.read.unread_button'), :controller => 'message', :action => 'mark', :message_id => @message.id, :mark => 'unread' %></td>
-    <td><%= link_to t('message.read.back_to_inbox'), :controller => 'message', :action => 'inbox', :display_name => @user.display_name %></td>
-  </tr>
-</table>
+<% if current_user == @message.recipient %>
+  <% content_for :heading do %>
+    <h2><%= h(@message.title) %></h2>
+  <% end %>
+
+  <div class='info-line clearfix'>
+    <%= user_thumbnail_tiny @message.sender %>
+    <%= link_to h(@message.sender.display_name), user_path(@message.sender) %></td>
+    <div class='right'>
+      <%= l @message.sent_on, :format => :friendly %>
+    </div>
+  </div>
+
+  <div class="richtext"><%= @message.body.to_html %></div>
+
+  <div class='message-buttons buttons'>
+    <%= button_to t('message.read.reply_button'), {:controller => 'message', :action => 'reply', :message_id => @message.id}, :class => 'reply-button' %>
+    <%= button_to t('message.read.unread_button'), {:controller => 'message', :action => 'mark', :message_id => @message.id, :mark => 'unread'}, :class => 'mark-unread-button' %>
+    <%= button_to t('message.read.delete_button'), {:controller => 'message', :action => 'delete', :message_id => @message.id}, :class => 'delete-button' %>
 
 <% else %>
 
-<h2><%= t'message.read.reading_your_sent_messages' %></h2>
+  <h2><%= h(@message.title) %></h2>
 
-<table>
-  <tr>
-    <th align="right"><%= t'message.read.to' %></th>
-    <td><%= link_to h(@message.recipient.display_name), :controller => 'user', :action => 'view', :display_name => @message.recipient.display_name %></td>
-  </tr>
-  <tr>
-    <th align="right"><%= t'message.read.subject' %></th>
-    <td><%= h(@message.title) %></td>
-  </tr>
-  <tr>
-    <th align="right"><%= t'message.read.date' %></th>
-    <td><%= l @message.sent_on %></td>
-  </tr>
-  <tr>
-    <th></th>
-    <td><%= htmlize(@message.body) %></td>
-  </tr>
-</table>
+  <div class='info-line clearfix'>
+    <%= user_thumbnail_tiny @message.recipient %>
+    <%= link_to h(@message.recipient.display_name), user_path(@message.recipient) %></td>
+    <div class='right'>
+      <%= l @message.sent_on, :format => :friendly %>
+    </div>
+  </div>
 
-<br />
+  <div class="richtext"><%= @message.body.to_html %></div>
 
-<table>
-  <tr>
-    <td><%= link_to t('message.read.back_to_outbox'), :controller => 'message', :action => 'outbox', :display_name => @user.display_name %></td>
-  </tr>
-</table>
+  <div class='message-buttons buttons'>
 
 <% end %>
+
+  <%= link_to t('message.read.back'), {:controller => 'message', :action => 'outbox', :display_name => current_user.display_name }, :class => "button deemphasize" %>
+  </div>