Merge remote-tracking branch 'upstream/pull/3258'

diff --git a/app/views/messages/inbox.html.erb b/app/views/messages/inbox.html.erb
index a27d7f15c8221d621703da9c5d6ecd4ce2a8d06d..4f2925b149e386f86fa4e07c372c8ec5152797fd 100644 (file)
--- a/app/views/messages/inbox.html.erb
+++ b/app/views/messages/inbox.html.erb
@@ -2,8 +2,11 @@
   <%= javascript_include_tag "messages" %>
 <% end %>
 
+<% content_for :heading_class, "pb-0" %>
+
 <% content_for :heading do %>
-  <ul class="nav nav-pills">
+  <h1><%= t("users.show.my messages") %></h1>
+  <ul class="nav nav-tabs">
     <li class="nav-item">
       <a class="nav-link active"><%= t ".my_inbox" %></a>
     </li>

diff --git a/app/views/messages/outbox.html.erb b/app/views/messages/outbox.html.erb
index 562bd9df1123cf27b6ccd72cb54d58b8aa8cd13e..0099b573d816a0d289ca4e65cf17ee52bc400ad5 100644 (file)
--- a/app/views/messages/outbox.html.erb
+++ b/app/views/messages/outbox.html.erb
@@ -2,8 +2,12 @@
   <%= javascript_include_tag "messages" %>
 <% end %>
 
+<% content_for :heading_class, "pb-0" %>
+
 <% content_for :heading do %>
-  <ul class="nav nav-pills">
+  <h1><%= t("users.show.my messages") %></h1>
+
+  <ul class="nav nav-tabs">
     <li class="nav-item">
       <%= link_to t(".my_inbox"), inbox_messages_path, :class => "nav-link" %>
     </li>

diff --git a/app/views/messages/show.html.erb b/app/views/messages/show.html.erb
index 3631467fd249277fa9c43af99c41ab53cd7826f7..74c61669fb21f2799ca6cdc1485b3ed2eaadf3ed 100644 (file)
--- a/app/views/messages/show.html.erb
+++ b/app/views/messages/show.html.erb
@@ -1,8 +1,8 @@
-<% if current_user == @message.recipient %>
-  <% content_for :heading do %>
-    <h2><%= @message.title %></h2>
-  <% end %>
+<% content_for :heading do %>
+  <h2><%= @message.title %></h2>
+<% end %>
 
+<% if current_user == @message.recipient %>
   <div class='info-line clearfix'>
     <%= user_thumbnail_tiny @message.sender %>
     <%= link_to @message.sender.display_name, user_path(@message.sender) %></td>
@@ -11,19 +11,15 @@
     </div>
   </div>
 
-  <div class="richtext"><%= @message.body.to_html %></div>
+  <div class="richtext text-break"><%= @message.body.to_html %></div>
 
   <div>
     <%= link_to t(".reply_button"), message_reply_path(@message), :class => "btn btn-primary" %>
     <%= link_to t(".unread_button"), message_mark_path(@message, :mark => "unread"), :method => "post", :class => "btn btn-primary" %>
-    <%= link_to t(".destroy_button"), message_path(@message), :method => "delete", :class => "btn btn-primary" %>
+    <%= link_to t(".destroy_button"), message_path(@message), :method => "delete", :class => "btn btn-danger" %>
     <%= link_to t(".back"), inbox_messages_path, :class => "btn btn-link" %>
   </div>
-
 <% else %>
-
-  <h2><%= @message.title %></h2>
-
   <div class='info-line clearfix'>
     <%= user_thumbnail_tiny @message.recipient %>
     <%= link_to @message.recipient.display_name, user_path(@message.recipient) %></td>
@@ -35,7 +31,7 @@
   <div class="richtext text-break"><%= @message.body.to_html %></div>
 
   <div>
+    <%= link_to t(".destroy_button"), message_path(@message), :method => "delete", :class => "btn btn-danger" %>
     <%= link_to t(".back"), outbox_messages_path, :class => "btn btn-link" %>
   </div>
-
 <% end %>