From a198cb1e279bfa13ad5e4285078b3f061d67ac18 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 14:04:05 +0200
Subject: [PATCH 1/9] Make permanent to avoid losing menu on page refreshes

---
 app/views/filters/_menu.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/filters/_menu.html.erb b/app/views/filters/_menu.html.erb
index 7b84cdd60..a117bdfa7 100644
--- a/app/views/filters/_menu.html.erb
+++ b/app/views/filters/_menu.html.erb
@@ -1,4 +1,4 @@
-<nav data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside">
+<nav data-controller="dialog" data-action="keydown.esc->dialog#close click@document->dialog#closeOnClickOutside" data-turbo-permanent>
   <%= render "filters/menu/button" %>
 
   <%= tag.dialog class: "fizzy-menu filter popup popup--animated panel margin-block-start-half",

From 10c9214d52e0b6f8af2054cf955d96ce7613e93d Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 14:04:42 +0200
Subject: [PATCH 2/9] Live updates for the timeline

---
 app/views/events/index.html.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/events/index.html.erb b/app/views/events/index.html.erb
index dcc0101c7..33f1df935 100644
--- a/app/views/events/index.html.erb
+++ b/app/views/events/index.html.erb
@@ -1,7 +1,7 @@
 <% @page_title = "Home" %>
 <% @header_class = "header--events" %>
 
-<%= turbo_stream_from :activity_summaries %>
+<%= render "filters/broadcasts", filter: @filter %>
 
 <% content_for :header do %>
   <%= render "filters/menu", user_filtering: @user_filtering %>

From 5f8457ad9d2a42f4f59c60f8a1a94d74b8db3d59 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 14:19:21 +0200
Subject: [PATCH 3/9] Don't broadcast updates of cards unless published

To prevent https://fizzy.37signals.com/5986089/cards/1631
---
 app/controllers/cards_controller.rb | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/app/controllers/cards_controller.rb b/app/controllers/cards_controller.rb
index 809428f91..cd8abfaa8 100644
--- a/app/controllers/cards_controller.rb
+++ b/app/controllers/cards_controller.rb
@@ -27,7 +27,9 @@ class CardsController < ApplicationController
   end
 
   def update
-    @card.update! card_params
+    suppressing_broadcasts_unless_published(@card) do
+      @card.update! card_params
+    end
 
     if @card.published?
       render_card_replacement
@@ -50,6 +52,14 @@ class CardsController < ApplicationController
       @card = Current.user.accessible_cards.find params[:id]
     end
 
+    def suppressing_broadcasts_unless_published(&block)
+      if @card.published?
+        yield
+      else
+        Collection.suppressing_turbo_broadcasts(&block)
+      end
+    end
+
     def card_params
       params.expect(card: [ :status, :title, :description, :image, tag_ids: [] ])
     end

From 0d6333f8baca4adace945cb9dfbe3801dbd7515d Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 15:07:52 +0200
Subject: [PATCH 4/9] Use frame-based pagination for the activity timeline

---
 app/controllers/cards_controller.rb          |  4 ++--
 app/helpers/events_helper.rb                 | 11 -----------
 app/helpers/pagination_helper.rb             | 13 +++++++++++++
 app/views/events/days/index.html.erb         |  4 ++++
 app/views/events/days/index.turbo_stream.erb |  7 -------
 app/views/events/index.html.erb              | 11 +++++++----
 6 files changed, 26 insertions(+), 24 deletions(-)
 create mode 100644 app/views/events/days/index.html.erb
 delete mode 100644 app/views/events/days/index.turbo_stream.erb

diff --git a/app/controllers/cards_controller.rb b/app/controllers/cards_controller.rb
index cd8abfaa8..526806879 100644
--- a/app/controllers/cards_controller.rb
+++ b/app/controllers/cards_controller.rb
@@ -52,8 +52,8 @@ class CardsController < ApplicationController
       @card = Current.user.accessible_cards.find params[:id]
     end
 
-    def suppressing_broadcasts_unless_published(&block)
-      if @card.published?
+    def suppressing_broadcasts_unless_published(card, &block)
+      if card.published?
         yield
       else
         Collection.suppressing_turbo_broadcasts(&block)
diff --git a/app/helpers/events_helper.rb b/app/helpers/events_helper.rb
index 0eecdb25f..9200f84b7 100644
--- a/app/helpers/events_helper.rb
+++ b/app/helpers/events_helper.rb
@@ -52,17 +52,6 @@ module EventsHelper
     tag.div class: "events__time-block", style: "grid-area: #{row}/#{col}", &
   end
 
-  def event_next_page_link(next_day)
-    if next_day
-      tag.div id: "next_page",
-        data: { controller: "fetch-on-visible",
-                fetch_on_visible_url_value: events_days_path(
-                  day: next_day.strftime("%Y-%m-%d"),
-                  **@filter.as_params
-                ) }
-    end
-  end
-
   def event_action_sentence(event)
     if event.action.comment_created?
       comment_event_action_sentence(event)
diff --git a/app/helpers/pagination_helper.rb b/app/helpers/pagination_helper.rb
index 8f2fc7988..3fdbe4923 100644
--- a/app/helpers/pagination_helper.rb
+++ b/app/helpers/pagination_helper.rb
@@ -35,6 +35,19 @@ module PaginationHelper
     end
   end
 
+  def day_timeline_pagination_frame_tag(day_timeline, &)
+    turbo_frame_tag day_timeline_pagination_frame_id_for(day_timeline.day), data: { timeline_target: "frame" }, role: "presentation", refresh: :morph, &
+  end
+
+  def day_timeline_pagination_frame_id_for(day)
+    "day-timeline-pagination-contents-#{day.strftime("%Y-%m-%d")}"
+  end
+
+  def day_timeline_pagination_link(day_timeline, filter)
+    link_to "Load more...", events_days_path(day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
+      data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" }
+  end
+
   private
     def pagination_list(name, tag_element: :div, paginate_on_scroll: false, **properties, &block)
       classes = properties.delete(:class)
diff --git a/app/views/events/days/index.html.erb b/app/views/events/days/index.html.erb
new file mode 100644
index 000000000..9529e8018
--- /dev/null
+++ b/app/views/events/days/index.html.erb
@@ -0,0 +1,4 @@
+<%= day_timeline_pagination_frame_tag @day_timeline do %>
+  <%= render "events/day", day_timeline: @day_timeline %>
+  <%= day_timeline_pagination_link(@day_timeline, @filter) %>
+<% end %>
diff --git a/app/views/events/days/index.turbo_stream.erb b/app/views/events/days/index.turbo_stream.erb
deleted file mode 100644
index cceb2d352..000000000
--- a/app/views/events/days/index.turbo_stream.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<%= turbo_stream.append :activity do %>
-  <%= render "events/day", day_timeline: @day_timeline %>
-<% end %>
-
-<%= turbo_stream.replace :next_page do %>
-  <%= event_next_page_link(@day_timeline.next_day) %>
-<% end %>
diff --git a/app/views/events/index.html.erb b/app/views/events/index.html.erb
index 33f1df935..dad76f7f2 100644
--- a/app/views/events/index.html.erb
+++ b/app/views/events/index.html.erb
@@ -14,7 +14,10 @@
   </h1>
 <% end %>
 
-<div class="events" id="activity">
-  <%= render "events/day", day_timeline: @day_timeline %>
-</div>
-<%= event_next_page_link(@day_timeline.next_day) %>
+<%= tag.div id: "activity", class: "events", data: { controller: "pagination", pagination_paginate_on_intersection_value: true } do %>
+  <%= day_timeline_pagination_frame_tag @day_timeline do %>
+    <%= render "events/day", day_timeline: @day_timeline %>
+
+    <%= day_timeline_pagination_link(@day_timeline, @filter) %>
+  <% end %>
+<% end %>

From bc90129af2e8bce2d227b7d3183c8e159c61808b Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 15:45:09 +0200
Subject: [PATCH 5/9] Use frame based pagination for the users timeline too

---
 app/controllers/users_controller.rb         | 10 +++++++++-
 app/views/events/event/_layout.html.erb     |  2 +-
 app/views/users/_activity_timeline.html.erb | 11 +++++++++++
 app/views/users/show.html.erb               |  5 +----
 4 files changed, 22 insertions(+), 6 deletions(-)
 create mode 100644 app/views/users/_activity_timeline.html.erb

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index d066632bd..11b793b17 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -24,7 +24,7 @@ class UsersController < ApplicationController
 
   def show
     @filter = Current.user.filters.new(creator_ids: [ @user.id ])
-    @day_timeline = Current.user.timeline_for(Time.current, filter: @filter)
+    @day_timeline = Current.user.timeline_for(day_param, filter: @filter)
   end
 
   def update
@@ -50,6 +50,14 @@ class UsersController < ApplicationController
       head :forbidden unless Current.user.can_change?(@user)
     end
 
+    def day_param
+      if params[:day].present?
+        Time.zone.parse(params[:day])
+      else
+        Time.current
+      end
+    end
+
     def user_params
       params.expect(user: [ :name, :email_address, :password, :avatar ])
     end
diff --git a/app/views/events/event/_layout.html.erb b/app/views/events/event/_layout.html.erb
index 48b25d0bd..63f8e696d 100644
--- a/app/views/events/event/_layout.html.erb
+++ b/app/views/events/event/_layout.html.erb
@@ -1,6 +1,6 @@
 <%= link_to event.notifiable_target,
       class: "event event--#{ event.action } #{ "golden-effect" if event.card.golden? } center-block flex flex-column full-width align-start justify-start position-relative",
-      style: "--card-color: #{ card.closed? ? "var(--color-card-complete)" : card.color };",
+      style: "--card-color: #{ card.closed? ? "var(--color-card-complete)" : card.color };", target: "_top",
       data: { related_element_target: "related",
             related_element_group_value: card.id,
             action: "mouseover->related-element#highlight mouseout->related-element#unhighlight" } do %>
diff --git a/app/views/users/_activity_timeline.html.erb b/app/views/users/_activity_timeline.html.erb
new file mode 100644
index 000000000..4bb6890de
--- /dev/null
+++ b/app/views/users/_activity_timeline.html.erb
@@ -0,0 +1,11 @@
+<div class="events margin-block-double" id="activity" data-controller="pagination" data-pagination-paginate-on-intersection-value="true">
+<%= day_timeline_pagination_frame_tag day_timeline do %>
+    <%= render "events/day", day_timeline: day_timeline %>
+
+    <%= link_to "Load more...", user_path(user, day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
+          data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" } %>
+  <% end %>
+</div>
+
+
+
diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index 6ae8f520c..9e27338c3 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -50,7 +50,4 @@
 
 <h1 class="font-weight-black txt-large margin-block-double"><%= "What #{ Current.user == @user ? "have you" : "has #{ @user.first_name }" } been up to?" %></h1>
 
-<div class="events margin-block-double" id="activity">
-  <%= render "events/day", day_timeline: @day_timeline %>
-</div>
-<%= event_next_page_link(@day_timeline.next_day) %>
+<%= render "users/activity_timeline", user: @user, day_timeline: @day_timeline, filter: @filter %>

From b986927e25ad86bbed227d85fb8f4bcf8ecb0aef Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 15:54:02 +0200
Subject: [PATCH 6/9] Add pagination to the list of filtered cards

---
 app/helpers/pagination_helper.rb | 9 +++++++++
 app/views/cards/index.html.erb   | 4 +++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/app/helpers/pagination_helper.rb b/app/helpers/pagination_helper.rb
index 3fdbe4923..388c244dc 100644
--- a/app/helpers/pagination_helper.rb
+++ b/app/helpers/pagination_helper.rb
@@ -35,6 +35,15 @@ module PaginationHelper
     end
   end
 
+  def with_automatic_pagination(name, page, **properties)
+    pagination_list name, paginate_on_scroll: true, **properties do
+      concat(pagination_frame_tag(name, page) do
+        yield
+        concat link_to_next_page(name, page, activate_when_observed: true)
+      end)
+    end
+  end
+
   def day_timeline_pagination_frame_tag(day_timeline, &)
     turbo_frame_tag day_timeline_pagination_frame_id_for(day_timeline.day), data: { timeline_target: "frame" }, role: "presentation", refresh: :morph, &
   end
diff --git a/app/views/cards/index.html.erb b/app/views/cards/index.html.erb
index 1e457c452..47d15dcb8 100644
--- a/app/views/cards/index.html.erb
+++ b/app/views/cards/index.html.erb
@@ -27,6 +27,8 @@
 
 <%= turbo_frame_tag :cards_container do %>
   <section class="cards cards--grid">
-    <%= render partial: "cards/display/preview", collection: @page.records, as: :card, locals: { draggable: true }, cached: ->(card) { cacheable_preview_parts_for(card) } %>
+    <%= with_automatic_pagination :cards_paginated_container, @page do %>
+      <%= render partial: "cards/display/preview", collection: @page.records, as: :card, locals: { draggable: true }, cached: ->(card) { cacheable_preview_parts_for(card) } %>
+    <% end %>
   </section>
 <% end %>

From dbc2f47f80af19fece86fb1366b652d0f1152dfd Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 15:58:27 +0200
Subject: [PATCH 7/9] Keep filter panel during page refreshes

---
 app/views/filters/_settings.html.erb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/views/filters/_settings.html.erb b/app/views/filters/_settings.html.erb
index 16b37ba6d..5642197fb 100644
--- a/app/views/filters/_settings.html.erb
+++ b/app/views/filters/_settings.html.erb
@@ -1,11 +1,12 @@
 <% cache user_filtering do %>
-  <%= tag.aside id: "filter-settings",
+  <%= tag.aside \
         class: class_names("filters flex align-center gap-half justify-center center margin-block-end", { "filters--expanded": user_filtering.expanded? }),
         data: {
           controller: "toggle-class filter-settings",
           toggle_class_toggle_class: "filters--expanded",
           filter_settings_filters_set_class: "filters--has-filters-set",
           filter_settings_no_filtering_url_value: no_filtering_url,
+          turbo_permanent: true,
           filter_settings_refresh_url_value: settings_refresh_path } do %>
     <%= form_with url: filter_url, method: :get, class: "", data: {
           controller: "form",

From 575d231650d5586310b1bd369cb9ec4416b5e59c Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 16:04:39 +0200
Subject: [PATCH 8/9] Live updates for the collection perma

---
 app/views/collections/show.html.erb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/views/collections/show.html.erb b/app/views/collections/show.html.erb
index ac7343885..96392f1d2 100644
--- a/app/views/collections/show.html.erb
+++ b/app/views/collections/show.html.erb
@@ -1,6 +1,8 @@
 <% @page_title = @collection.name %>
 <% turbo_exempts_page_from_cache %>
 
+<%= turbo_stream_from @collection %>
+
 <% content_for :head do %>
   <%= tag.meta property: "og:title", content: @page_title %>
   <%= tag.meta property: "og:description", content: Account.sole.name %>

From 44c3910699cf1f18efe6de8e40373ed2bd844b02 Mon Sep 17 00:00:00 2001
From: Jorge Manrubia <jorge@hey.com>
Date: Thu, 2 Oct 2025 16:10:00 +0200
Subject: [PATCH 9/9] Don't render next link unless there is more activity

---
 app/helpers/pagination_helper.rb            | 6 ++++--
 app/views/users/_activity_timeline.html.erb | 8 +++++---
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/app/helpers/pagination_helper.rb b/app/helpers/pagination_helper.rb
index 388c244dc..ee665e98a 100644
--- a/app/helpers/pagination_helper.rb
+++ b/app/helpers/pagination_helper.rb
@@ -53,8 +53,10 @@ module PaginationHelper
   end
 
   def day_timeline_pagination_link(day_timeline, filter)
-    link_to "Load more...", events_days_path(day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
-      data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" }
+    if day_timeline.next_day
+      link_to "Load more...", events_days_path(day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
+        data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" }
+    end
   end
 
   private
diff --git a/app/views/users/_activity_timeline.html.erb b/app/views/users/_activity_timeline.html.erb
index 4bb6890de..1c3b84d56 100644
--- a/app/views/users/_activity_timeline.html.erb
+++ b/app/views/users/_activity_timeline.html.erb
@@ -1,9 +1,11 @@
 <div class="events margin-block-double" id="activity" data-controller="pagination" data-pagination-paginate-on-intersection-value="true">
-<%= day_timeline_pagination_frame_tag day_timeline do %>
+  <%= day_timeline_pagination_frame_tag day_timeline do %>
     <%= render "events/day", day_timeline: day_timeline %>
 
-    <%= link_to "Load more...", user_path(user, day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
-          data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" } %>
+    <% if day_timeline.next_day %>
+      <%= link_to "Load more...", user_path(user, day: day_timeline.next_day.strftime("%Y-%m-%d"), **filter.as_params),
+            data: { frame: day_timeline_pagination_frame_id_for(day_timeline.next_day), pagination_target: "paginationLink" } %>
+    <% end %>
   <% end %>
 </div>