Change mark message buttons visibility with hidden attr

diff --git a/app/assets/javascripts/messages.js b/app/assets/javascripts/messages.js
index 8c30f9b34a726b2c399d2ab5eb473f8bb2029cb7..d34ffb46662bfa8caa1aed64b90271b146559d8c 100644 (file)
--- a/app/assets/javascripts/messages.js
+++ b/app/assets/javascripts/messages.js
@@ -27,6 +27,8 @@ $(document).ready(function () {
   function updateReadState(target, isRead) {
     $(target).closest("tr")
       .toggleClass("inbox-row", isRead)
-      .toggleClass("inbox-row-unread", !isRead);
+      .toggleClass("inbox-row-unread", !isRead)
+      .find(".inbox-mark-unread button").prop("hidden", !isRead).end()
+      .find(".inbox-mark-read button").prop("hidden", isRead);
   }
 });

diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index 8e29424c278aa7c71789cf5ede7c2212668e9a59..f7192a68eaf5ac34cd9a4d31e77233de19e4a8fc 100644 (file)
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -1141,14 +1141,6 @@ tr.turn:hover {
   }
 }
 
-.inbox-row .inbox-mark-read {
-  display: none !important;
-}
-
-.inbox-row-unread .inbox-mark-unread {
-  display: none !important;
-}
-
 .search_form {
   background-color: $lightgrey;
 

diff --git a/app/views/messages/_message_summary.html.erb b/app/views/messages/_message_summary.html.erb
index e93b7fa900caad4c3b6bedb271ec50f9c4eaf7a8..dfcb3b02a72549743672e7837697d1d7245fd117 100644 (file)
--- a/app/views/messages/_message_summary.html.erb
+++ b/app/views/messages/_message_summary.html.erb
@@ -3,8 +3,8 @@
   <td class="inbox-subject"><%= link_to message_summary.title, message_path(message_summary) %></td>
   <td class="text-nowrap"><%= l message_summary.sent_on, :format => :friendly %></td>
   <td class="text-nowrap text-end">
-    <%= button_to t(".unread_button"), message_mark_path(message_summary, :mark => "unread"), :remote => true, :class => "btn btn-sm btn-primary", :form_class => "d-inline-block inbox-mark-unread" %>
-    <%= button_to t(".read_button"), message_mark_path(message_summary, :mark => "read"), :remote => true, :class => "btn btn-sm btn-primary", :form_class => "d-inline-block inbox-mark-read" %>
+    <%= button_to t(".unread_button"), message_mark_path(message_summary, :mark => "unread"), :remote => true, :class => "btn btn-sm btn-primary", :form_class => "d-inline-block inbox-mark-unread", :hidden => !message_summary.message_read? %>
+    <%= button_to t(".read_button"), message_mark_path(message_summary, :mark => "read"), :remote => true, :class => "btn btn-sm btn-primary", :form_class => "d-inline-block inbox-mark-read", :hidden => message_summary.message_read? %>
     <%= button_to t(".destroy_button"), message_path(message_summary, :referer => request.fullpath), :method => :delete, :remote => true, :class => "btn btn-sm btn-danger", :form_class => "d-inline-block inbox-destroy" %>
   </td>
 </tr>