From d455537392f58e392211e2e99e33e9e2fd21c339 Mon Sep 17 00:00:00 2001
From: Andy Smith <andy@37signals.com>
Date: Mon, 30 Jun 2025 12:56:56 -0500
Subject: [PATCH] Remove home button

---
 app/assets/stylesheets/header.css             |  9 ++++++++
 app/views/cards/index.html.erb                | 21 +++++--------------
 app/views/cards/index/_header.html.erb        |  4 ----
 .../index/engagement/_considering.html.erb    |  9 ++++++++
 app/views/filters/_settings.html.erb          |  4 ++--
 5 files changed, 25 insertions(+), 22 deletions(-)

diff --git a/app/assets/stylesheets/header.css b/app/assets/stylesheets/header.css
index 29cb298b4..0fb0ce6fe 100644
--- a/app/assets/stylesheets/header.css
+++ b/app/assets/stylesheets/header.css
@@ -33,4 +33,13 @@
     margin: 0 auto;
     text-align: center;
   }
+
+  .header__title--actions {
+    --btn-size: 2rem;
+    --btn-icon-size: 1rem;
+
+    display: flex;
+    align-items: center;
+    gap: 0.33ch;
+  }
 }
diff --git a/app/views/cards/index.html.erb b/app/views/cards/index.html.erb
index 0f302d75f..9c0cb7bac 100644
--- a/app/views/cards/index.html.erb
+++ b/app/views/cards/index.html.erb
@@ -7,21 +7,8 @@
 <% end %>
 
 <header class="header">
-  <div class="header__actions header__actions--start">
-    <% if collection = @filter.single_collection || Current.user.collections.first %>
-      <%= button_to collection_cards_path(collection), method: :post, class: "btn btn--link txt-small", form_class: "full-width" do %>
-        <%= icon_tag "add" %>
-        <span>Add a card</span>
-      <% end %>
-    <% end %>
-  </div>
-
-  <h1 class="header__title overflow-ellipsis" style="view-transistion-name: card-collection-title">
-    <span><%= filter_title @filter %></span>
-  </h1>
-
-  <div class="header__actions header__actions--end">
-    <%= render "filters/settings", filter: @filter %>
+  <h1 class="header__title header__title--actions" style="view-transistion-name: card-collection-title">
+    <span class="overflow-ellipsis"><%= filter_title @filter %></span>
     <% if collection = @filter.single_collection %>
       <%= access_involvement_advance_button collection, Current.user %>
 
@@ -30,9 +17,11 @@
         <span class="for-screen-reader">Settings for <%= collection.name %></span>
       <% end %>
     <% end %>
-  </div>
+  </h1>
 </header>
 
+<%= render "filters/settings", filter: @filter %>
+
 <%= tag.div data: {
       controller: "drag-and-drop",
       drag_and_drop_dragged_item_class: "drag-and-drop__dragged-item",
diff --git a/app/views/cards/index/_header.html.erb b/app/views/cards/index/_header.html.erb
index b4c93cd71..a9e0a9229 100644
--- a/app/views/cards/index/_header.html.erb
+++ b/app/views/cards/index/_header.html.erb
@@ -1,7 +1,4 @@
 <nav class="header">
-  <div class="header__actions header__actions--start">
-    <%= link_to_home %>
-  </div>
   <header class="flex flex-inline flex-column center flex-shrink">
     <div class="card-filter--collection flex-inline flex-column align-center center position-relative" data-controller="dialog" data-action="click@document->dialog#closeOnClickOutside keydown.esc->dialog#close">
       <%= tag.button class: "fizzy-menu input input--select flex align-center txt-normal shadow",
@@ -52,5 +49,4 @@
       <% end %>
     </div>
   </header>
-  <div class="header__actions header__actions--end"></div>
 </nav>
diff --git a/app/views/cards/index/engagement/_considering.html.erb b/app/views/cards/index/engagement/_considering.html.erb
index fb4a70a25..311303b68 100644
--- a/app/views/cards/index/engagement/_considering.html.erb
+++ b/app/views/cards/index/engagement/_considering.html.erb
@@ -3,6 +3,15 @@
     Considering
   </h2>
 
+  <% if collection = filter.single_collection || Current.user.collections.first %>
+    <%= button_to collection_cards_path(collection), method: :post, class: "card card--new-card-button btn", form_class: "full-width" do %>
+      <div class="gap-half justify-center txt-xx-large pad-block margin-block-half">
+        <%= icon_tag "add" %>
+        <span>Add a new card</span>
+      </div>
+    <% end %>
+  <% end %>
+
   <% if page.used? %>
     <%= render partial: "cards/display/preview", collection: page.records, as: :card, locals: { draggable: true }, cached: true %>
 
diff --git a/app/views/filters/_settings.html.erb b/app/views/filters/_settings.html.erb
index ecc39e45d..d1b582631 100644
--- a/app/views/filters/_settings.html.erb
+++ b/app/views/filters/_settings.html.erb
@@ -1,4 +1,4 @@
-<div class="filters flex align-center gap-half margin-none-block-end margin-block-start-half justify-center center">
+<aside class="filters flex align-center gap-half justify-center center margin-block-end">
   <div class="flex-inline center align-center gap-half">
     <%= render "filters/tags", filter: filter %>
     <%= render "filters/assignees", filter: filter %>
@@ -18,4 +18,4 @@
       <% end %>
     <% end %>
   </div>
-</div>
+</aside>