From a1e650fb501658b7e84f4bc0192f7047c9159205 Mon Sep 17 00:00:00 2001
From: Jason Zimdars <jz@37signals.com>
Date: Wed, 1 Oct 2025 13:58:15 -0500
Subject: [PATCH] Remove filtering checkboxes, replace with icon; modify icon

---
 app/assets/images/collection.svg                   |  2 +-
 .../filters/menu/collections/_all_option.html.erb  |  9 +--------
 .../filters/menu/collections/_collection.html.erb  | 14 +-------------
 3 files changed, 3 insertions(+), 22 deletions(-)

diff --git a/app/assets/images/collection.svg b/app/assets/images/collection.svg
index 5d6249839..cdbe7d4da 100644
--- a/app/assets/images/collection.svg
+++ b/app/assets/images/collection.svg
@@ -1 +1 @@
-<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><path d="m3 5h8v2h-8zm10 0h8v2h-8zm-10 4h8v2h-8zm10 0h8v2h-8zm-10 4h8v2h-8zm10 0h8v2h-8zm-10 4h8v2h-8z"/></svg>
\ No newline at end of file
+<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><path d="m21.8 0h-19.6c-1.2 0-2.2 1-2.2 2.2v19.5c0 1.2 1 2.2 2.2 2.2h19.5c1.2 0 2.2-1 2.2-2.2v-19.5c0-1.2-1-2.2-2.2-2.2zm-10.8 19h-8v-2h8zm0-4h-8v-2h8zm0-4h-8v-2h8zm0-4h-8v-2h8zm10 8h-8v-2h8zm0-4h-8v-2h8zm0-4h-8v-2h8z"/></svg>
\ No newline at end of file
diff --git a/app/views/filters/menu/collections/_all_option.html.erb b/app/views/filters/menu/collections/_all_option.html.erb
index 4c933510f..288877618 100644
--- a/app/views/filters/menu/collections/_all_option.html.erb
+++ b/app/views/filters/menu/collections/_all_option.html.erb
@@ -2,14 +2,7 @@
 
 <% if Current.user.collections.many? %>
   <li class="popup__item" data-filter-target="item" data-navigable-list-target="item" id="filter-collection-all">
-    <%= link_to clear_url_path, class: "popup__radio btn" do %>
-      <%= check_box_tag "filter_collections", nil, user_filtering.collections.blank?, class: "form-checkbox" %>
-      <span class="for-screen-reader">
-        See everything in all collections
-      </span>
-      <%= icon_tag "check", size: 18, class: "checked" %>
-    <% end %>
-
+    <%= icon_tag "collection", class: "popup__icon" %>
     <%= link_to clear_url_path, class: "popup__btn btn" do %>
       <span class="overflow-ellipsis">All collections</span>
     <% end %>
diff --git a/app/views/filters/menu/collections/_collection.html.erb b/app/views/filters/menu/collections/_collection.html.erb
index 48a8ac223..b4c1e1cc3 100644
--- a/app/views/filters/menu/collections/_collection.html.erb
+++ b/app/views/filters/menu/collections/_collection.html.erb
@@ -1,17 +1,5 @@
 <li class="popup__item" data-filter-target="item" data-navigable-list-target="item" id="filter-collection-<%= collection.id %>">
-  <label class="popup__radio btn">
-    <%= form.check_box "collection_ids[]", {
-          checked: user_filtering.filter.collections.include?(collection),
-          data: { action: "change->form#submit" },
-          include_hidden: false,
-    }, collection.id %>
-
-    <span class="for-screen-reader">
-      Toggle filter for <%= collection.name %>
-    </span>
-    <%= icon_tag "check", size: 18, class: "checked" %>
-  </label>
-
+  <%= icon_tag "collection", class: "popup__icon" %>
   <%= link_to collection_path(collection), class: "popup__btn btn" do %>
     <span class="overflow-ellipsis"><%= collection.name %></span>
   <% end %>