From ac3307ee230babd9e52ae620bdc4ef199782d822 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@37signals.com>
Date: Wed, 16 Apr 2025 16:23:05 +0200
Subject: [PATCH] Allow for the footer frames to be hidden on the notifications
 page

Otherwise you will have multiple notification dom elements with the same
id.
---
 app/views/layouts/application.html.erb | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index e886c519f..fd857fafe 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -46,10 +46,12 @@
     <footer id="footer">
       <%= yield :footer %>
 
-      <% if Current.user %>
-        <%= render "notifications/tray" %>
-        <%= render "my/pins/tray" %>
-        <%= turbo_frame_tag "terminal", src: terminal_path, data: { turbo_permanent: true } %>
+      <% if Current.user && !@hide_footer_frames %>
+        <div id="footer_frames" data-turbo-permanent="true">
+          <%= render "notifications/tray" %>
+          <%= render "my/pins/tray" %>
+          <%= turbo_frame_tag "terminal", src: terminal_path %>
+        </div>
       <% end %>
     </footer>