From dcf26d69815117ff3365246a044c9e1676d23757 Mon Sep 17 00:00:00 2001
From: Kevin McConnell <kevin@37signals.com>
Date: Thu, 6 Feb 2025 14:41:05 +0000
Subject: [PATCH] Hide these buttons when there's no current bucket

These depend on having a bucket to link to, but in the case of an "All
buckets" view, we don't have one. So we can hide them in that case.
---
 app/views/bubbles/index.html.erb | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/app/views/bubbles/index.html.erb b/app/views/bubbles/index.html.erb
index 24367f146..c9b37fe45 100644
--- a/app/views/bubbles/index.html.erb
+++ b/app/views/bubbles/index.html.erb
@@ -5,10 +5,11 @@
 
 <% content_for :header do %>
   <nav class="align-start">
-    <% bucket = @filter.buckets.first %>
-    <%= link_to edit_bucket_path(bucket), class: "btn borderless flex-item-justify-start" do %>
-      <%= image_tag "settings.svg", aria: { hidden: true }, size: 24 %>
-      <span class="for-screen-reader">Settings for <%= bucket.name %></span>
+    <% if bucket = @filter.buckets.first %>
+      <%= link_to edit_bucket_path(bucket), class: "btn borderless flex-item-justify-start" do %>
+        <%= image_tag "settings.svg", aria: { hidden: true }, size: 24 %>
+        <span class="for-screen-reader">Settings for <%= bucket.name %></span>
+      <% end %>
     <% end %>
 
     <header class="flex flex-column center">
@@ -19,9 +20,11 @@
       <%= render "bubbles/filters", filter: @filter %>
     </header>
 
-    <%= button_to bucket_bubbles_path(bucket), method: :post, class: "btn borderless", form_class: "flex-item-justify-end" do %>
-      <%= image_tag "add.svg", aria: { hidden: true }, size: 24 %>
-      <span class="for-screen-reader">Create a new bubble</span>
+    <% if bucket = @filter.buckets.first %>
+      <%= button_to bucket_bubbles_path(bucket), method: :post, class: "btn borderless", form_class: "flex-item-justify-end" do %>
+        <%= image_tag "add.svg", aria: { hidden: true }, size: 24 %>
+        <span class="for-screen-reader">Create a new bubble</span>
+      <% end %>
     <% end %>
   </nav>
 <% end %>