Fix picture archive select-all feature

AlchemyCMS · Dec 14, 2023 · d422fc5 · d422fc5
1 parent 474c72b
commit d422fc5
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 5 deletions.
diff --git a/app/assets/stylesheets/alchemy/archive.scss b/app/assets/stylesheets/alchemy/archive.scss
@@ -30,7 +30,6 @@
   border-bottom: 1px solid $default-border-color;
   margin-bottom: 4 * $default-padding;
   padding: 4 * $default-padding;
-  display: none;
 
   .button {
     vertical-align: middle;
@@ -89,14 +88,23 @@
 
   &:hover,
   &:focus-within {
-    box-shadow: 0 0 4px 2px $default-border-color;
-
     .picture_tool,
     .picture_tags {
       visibility: visible;
       opacity: 1;
     }
   }
+
+  &:hover,
+  &:focus-within,
+  &.active {
+    box-shadow: 0 0 4px 2px $default-border-color;
+
+    .picture_tool.select {
+      visibility: visible;
+      opacity: 1;
+    }
+  }
 }
 
 #pictures,

diff --git a/app/views/alchemy/admin/pictures/_archive.html.erb b/app/views/alchemy/admin/pictures/_archive.html.erb
@@ -9,7 +9,7 @@
 </div>
 
 <%= form_tag delete_multiple_admin_pictures_path, method: :delete do %>
-  <div class="selected_item_tools">
+  <div class="selected_item_tools hidden">
     <h3><%= Alchemy.t(:edit_selected_pictures) %></h3>
     <%= link_to(
       render_icon(:edit, size: '1x') + Alchemy.t("Edit"),

diff --git a/app/views/alchemy/admin/pictures/index.html.erb b/app/views/alchemy/admin/pictures/index.html.erb
@@ -90,8 +90,13 @@
     $(function() {
       Alchemy.pictureSelector();
       $('#select_all_pictures').on('click', function(e) {
-        $('.picture_tool.select :checkbox').prop('checked', true).change();
+        $(this).toggleClass("active")
+        $('.picture_tool.select :checkbox')
+          .prop('checked', function(_i, val) { return !val })
+          .closest(".picture_thumbnail")
+          .toggleClass("active");
         e.preventDefault;
+        $(".selected_item_tools").toggleClass("hidden")
         return false;
       });
       $('.thumbnail_background').click(function(event) {