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

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 61c825fca81c818f9125c892ecf937f849540701..cf616983e29f708b68bbf32f8ed90980d578974d 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -75,7 +75,10 @@ time[title] {
   .table-secondary {
     --bs-table-bg: rgb(var(--bs-secondary-rgb), .25);
   }
-  .table-primary, .table-secondary {
+  .table-success {
+    --bs-table-bg: rgb(var(--bs-success-rgb), .25);
+  }
+  .table-primary, .table-secondary, .table-success {
     --bs-table-color: initial;
     border-color: inherit;
   }
@@ -812,14 +815,6 @@ tr.turn {
   }
 }
 
-/* Rules for messages pages */
-
-.messages {
-  .inbox-row-unread td {
-    background: #CBEEA7;
-  }
-}
-
 /* Rules for user images */
 
 img.user_image {

diff --git a/app/assets/stylesheets/parameters.scss b/app/assets/stylesheets/parameters.scss
index 3d66db5fbe65cb013289a6b61d6d60978794386f..719f5ad3b068098fb597db611770d0891c3b0ca9 100644 (file)
--- a/app/assets/stylesheets/parameters.scss
+++ b/app/assets/stylesheets/parameters.scss
@@ -19,4 +19,6 @@ $link-hover-color: #24d;
 $link-decoration: none;
 $link-hover-decoration: underline;
 
+$table-border-factor: .1;
+
 $enable-negative-margins: true;

diff --git a/app/views/messages/_message_summary.html.erb b/app/views/messages/_message_summary.html.erb
index 02c60a752afa72ecaa085fc24bff339a0e2a003d..1a7a0c839f59e53db9413b16ae15eb1b0a101f1d 100644 (file)
--- a/app/views/messages/_message_summary.html.erb
+++ b/app/views/messages/_message_summary.html.erb
@@ -1,4 +1,4 @@
-<tr id="inbox-<%= message.id %>" class="message-summary inbox-row<%= "-unread" unless message.message_read? %>">
+<%= tag.tr(:id => "inbox-#{message.id}", :class => { "message-summary" => true, "table-success" => !message.message_read? }) do %>
   <td><%= link_to message.sender.display_name, user_path(message.sender) %></td>
   <td><%= link_to message.title, message_path(message) %></td>
   <td class="text-nowrap"><%= l message.sent_on, :format => :friendly %></td>
@@ -12,4 +12,4 @@
       <% end %>
     </div>
   </td>
-</tr>
+<% end %>

diff --git a/app/views/messages/_sent_message_summary.html.erb b/app/views/messages/_sent_message_summary.html.erb
index 82bbafc16cb0e964676eda9d6906ab63a95603dc..cd156aa47503aaf92236b18c480e7849a1cb851f 100644 (file)
--- a/app/views/messages/_sent_message_summary.html.erb
+++ b/app/views/messages/_sent_message_summary.html.erb
@@ -1,4 +1,4 @@
-<tr id="outbox-<%= message.id %>" class="message-summary inbox-row">
+<%= tag.tr(:id => "outbox-#{message.id}", :class => { "message-summary" => true }) do %>
   <td><%= link_to message.recipient.display_name, user_path(message.recipient) %></td>
   <td><%= link_to message.title, message_path(message) %></td>
   <td class="text-nowrap"><%= l message.sent_on, :format => :friendly %></td>
@@ -7,4 +7,4 @@
       <%= button_to t(".destroy_button"), message_path(message, :referer => request.fullpath), :method => :delete, :class => "btn btn-sm btn-danger", :form => { :data => { :turbo => true }, :class => "destroy-message" } %>
     </div>
   </td>
-</tr>
+<% end %>

diff --git a/test/controllers/messages_controller_test.rb b/test/controllers/messages_controller_test.rb
index 40581993fc6a7b6d648e9cc2928a863fbfe36959..dfec6346f19f2c3a42783ad9bc4aba5ae5bc832e 100644 (file)
--- a/test/controllers/messages_controller_test.rb
+++ b/test/controllers/messages_controller_test.rb
@@ -339,9 +339,9 @@ class MessagesControllerTest < ActionDispatch::IntegrationTest
     get inbox_messages_path
     assert_response :success
     assert_template "inbox"
-    assert_select ".content-inner > table", :count => 1 do
-      assert_select "tr", :count => 2
-      assert_select "tr#inbox-#{read_message.id}.inbox-row", :count => 1 do
+    assert_select ".content-inner > table.messages-table > tbody", :count => 1 do
+      assert_select "tr", :count => 1
+      assert_select "tr#inbox-#{read_message.id}", :count => 1 do
         assert_select "a[href='#{user_path read_message.sender}']", :text => read_message.sender.display_name
         assert_select "a[href='#{message_path read_message}']", :text => read_message.title
       end
@@ -365,9 +365,9 @@ class MessagesControllerTest < ActionDispatch::IntegrationTest
     get outbox_messages_path
     assert_response :success
     assert_template "outbox"
-    assert_select ".content-inner > table", :count => 1 do
-      assert_select "tr", :count => 2
-      assert_select "tr.inbox-row", :count => 1 do
+    assert_select ".content-inner > table.messages-table > tbody", :count => 1 do
+      assert_select "tr", :count => 1
+      assert_select "tr#outbox-#{message.id}", :count => 1 do
         assert_select "a[href='#{user_path message.recipient}']", :text => message.recipient.display_name
         assert_select "a[href='#{message_path message}']", :text => message.title
       end