From 30fad13dc2102bdf6b55921aacfafc4222d45a00 Mon Sep 17 00:00:00 2001
From: Jan Suchal <jan.suchal@gmail.com>
Date: Sat, 16 Nov 2024 20:13:42 +0100
Subject: [PATCH 1/2] Fix cache behaviour for visited links

---
 .env.test                                                 | 4 ++++
 app/components/message_threads_table_component.html.erb   | 2 +-
 .../message_threads_table_row_component.html.erb          | 6 +++---
 app/javascript/controllers/index.js                       | 3 +++
 app/javascript/controllers/visited_links_controller.js    | 8 ++++++++
 5 files changed, 19 insertions(+), 4 deletions(-)
 create mode 100644 app/javascript/controllers/visited_links_controller.js

diff --git a/.env.test b/.env.test
index 9b5f06bf3..d2432a2a1 100644
--- a/.env.test
+++ b/.env.test
@@ -11,5 +11,9 @@ ZwIDAQAB
 ACTIVE_RECORD_ENCRYPTION_PRIMARY_KEY=1
 ACTIVE_RECORD_ENCRYPTION_DETERMINISTIC_KEY=2
 ACTIVE_RECORD_ENCRYPTION_KEY_DERIVATION_SALT=3
+
+DB_HOST=localhost
+DB_USER=postgres
+DB_PASSWORD=password
 GROVER_NO_SANDBOX=true  # must be true for running chromium as root in Docker container
 PDF_DISPLAY_URL=
diff --git a/app/components/message_threads_table_component.html.erb b/app/components/message_threads_table_component.html.erb
index 68f60cd92..dd789a1c4 100644
--- a/app/components/message_threads_table_component.html.erb
+++ b/app/components/message_threads_table_component.html.erb
@@ -2,7 +2,7 @@
   <div class="flex flex-col justify-stretch items-stretch sm:rounded-md bg-white sm:border sm:border-gray-200" data-controller="form all-checkboxes">
     <%= render MessageThreadsBulkActionsComponent.new(ids: [], filter: @filter, filter_subscription: @filter_subscription, signable: Current.user.signer?) %>
     <%= form_with url: bulk_actions_message_threads_path, data: { "form-target": "form", "all-checkboxes-target": "form" } do %>
-      <ul role="list" id="message_threads" class="divide-y divide-gray-100">
+      <ul role="list" id="message_threads" data-controller="visited-links" class="divide-y divide-gray-100">
         <% message_threads.each do |message_thread| %>
           <%= message_thread %>
         <% end %>
diff --git a/app/components/message_threads_table_row_component.html.erb b/app/components/message_threads_table_row_component.html.erb
index f4ee8b257..b70e69e9f 100644
--- a/app/components/message_threads_table_row_component.html.erb
+++ b/app/components/message_threads_table_row_component.html.erb
@@ -2,11 +2,11 @@
   <div class="hidden sm:flex pr-2 md:gap-4 md:pr-4 items-center justify-start mb-auto h-8">
     <%= check_box_tag("message_thread_ids[]", @message_thread.id, false, { id: "message_thread_#{@message_thread.id}", class: "h-4 w-4 rounded border-gray-300 text-blue-500 focus:ring-0 hidden sm:block my-auto", type: "checkbox", data: { action: "form#send" } }) %>
   </div>
-  <%= link_to @message_thread, class: "flex flex-col md:flex-row md:flex-nowrap justify-between grow gap-2" do %>
+  <%= link_to @message_thread, class: class_names("flex flex-col md:flex-row md:flex-nowrap justify-between grow gap-2", :group, visited: @message_thread.all_read), data: {action: 'visited-links#visit'} do %>
     <div class="flex flex-col gap-2">
       <div class="flex flex-wrap grow gap-x-4 gap-y-1">
         <div class="flex items-center sm:min-h-[32px]">
-          <span title="<%= @message_thread.title %>" class="inline text-sm sm:text-lg text-gray-900 <%= "font-semibold" unless @message_thread.all_read %> [word-break:break-word]">
+          <span title="<%= @message_thread.title %>" class="inline text-sm sm:text-lg text-gray-900 group-[:not(.visited)]:font-semibold [word-break:break-word]">
               <%= @message_thread.title %>
           </span>
         </div>
@@ -52,7 +52,7 @@
     </div>
     <div class="hidden md:flex gap-x-4 items-center">
       <div class="flex flex-col gap-y-1 text-right whitespace-nowrap">
-        <time class="text-lg text-gray-900 <%= "font-semibold" unless @message_thread.all_read %>" datetime="<%= @message_thread.last_message_delivered_at %>">
+        <time class="text-lg text-gray-900 group-[:not(.visited)]:font-semibold" datetime="<%= @message_thread.last_message_delivered_at %>">
           <%= helpers.nice_datetime(@message_thread.last_message_delivered_at) %>
         </time>
         <%#= render Common::TagComponent.new("2 nové správy") %>
diff --git a/app/javascript/controllers/index.js b/app/javascript/controllers/index.js
index 7f6ed8ed9..d670cfc75 100644
--- a/app/javascript/controllers/index.js
+++ b/app/javascript/controllers/index.js
@@ -39,3 +39,6 @@ application.register("turbo-content", TurboContentController)
 
 import ContentWithSeparator from "./content_with_separator"
 application.register("content-with-separator", ContentWithSeparator)
+
+import VisitedLinks from "./visited_links_controller"
+application.register("visited-links", VisitedLinks)
diff --git a/app/javascript/controllers/visited_links_controller.js b/app/javascript/controllers/visited_links_controller.js
new file mode 100644
index 000000000..c2ace28a0
--- /dev/null
+++ b/app/javascript/controllers/visited_links_controller.js
@@ -0,0 +1,8 @@
+import {Controller} from "@hotwired/stimulus"
+
+// Connects to data-controller="visited-links"
+export default class extends Controller {
+    visit(event) {
+        event.target.closest("a").classList.add("visited");
+    }
+}

From 2af3b444c4ad1041ac14ca2ffbc8f48bd355058c Mon Sep 17 00:00:00 2001
From: Jan Suchal <jan.suchal@gmail.com>
Date: Sat, 16 Nov 2024 20:16:04 +0100
Subject: [PATCH 2/2] Remove test junk

---
 .env.test | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/.env.test b/.env.test
index d2432a2a1..d32d7f7b9 100644
--- a/.env.test
+++ b/.env.test
@@ -12,8 +12,5 @@ ACTIVE_RECORD_ENCRYPTION_PRIMARY_KEY=1
 ACTIVE_RECORD_ENCRYPTION_DETERMINISTIC_KEY=2
 ACTIVE_RECORD_ENCRYPTION_KEY_DERIVATION_SALT=3
 
-DB_HOST=localhost
-DB_USER=postgres
-DB_PASSWORD=password
 GROVER_NO_SANDBOX=true  # must be true for running chromium as root in Docker container
 PDF_DISPLAY_URL=