From 6f55763452e227e17d8d1f17d9a88d3e0cf44d91 Mon Sep 17 00:00:00 2001
From: djanelle-mit <djanelle@mit.edu>
Date: Fri, 10 Apr 2026 13:01:32 -0400
Subject: [PATCH 1/3] Adding tab tracking to event names

---
 app/views/layouts/_site_header.html.erb     |  4 ++--
 app/views/layouts/application.html.erb      |  4 ++--
 app/views/search/_form.html.erb             | 22 +++++++++++----------
 app/views/search/_results_callouts.html.erb |  2 +-
 app/views/search/_results_sidebar.html.erb  |  2 +-
 app/views/search/results.html.erb           |  2 +-
 app/views/tacos/analyze.html.erb            |  2 +-
 app/views/thirdiron/libkey.html.erb         |  8 ++++----
 8 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/app/views/layouts/_site_header.html.erb b/app/views/layouts/_site_header.html.erb
index 9c03b0d7..ed0096d1 100644
--- a/app/views/layouts/_site_header.html.erb
+++ b/app/views/layouts/_site_header.html.erb
@@ -1,4 +1,4 @@
-<div class="institute-bar" data-matomo-seen="Navigation, Header Links Seen" data-matomo-click="Navigation, Header Links Engaged, Link: {{getElementText}}">
+<div class="institute-bar" data-matomo-seen="Navigation, Header Links Seen, Tab: {{getActiveTabName}}" data-matomo-click="Navigation, Header Links Engaged, Link: {{getElementText}}">
   <div class="wrapper">
     <a class="link-logo-mit" href="https://www.mit.edu" ><span class="sr">MIT Logo</span>
       <img src="https://cdn.libraries.mit.edu/files/branding/local/mit_logo_std_rgb_white.svg" height="24" alt="MIT logo" >
@@ -26,5 +26,5 @@
       <% end %>
     </header>
     <%= render partial: 'search/form' unless Feature.enabled?(:geodata) %>
-  </div>  
+  </div>
 </div>
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index 34a54c4e..e329120b 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -11,7 +11,7 @@
       <%= render partial: "layouts/global_alert" %>
 
       <%= render partial: "layouts/site_header" %>
-      
+
       <div class="wrap-outer-content layout-band">
         <div class="wrap-content">
           <%= render partial: "layouts/flash" %>
@@ -22,7 +22,7 @@
         </div>
       </div>
 
-      <footer data-matomo-seen="Navigation, Footer Links Seen" data-matomo-click="Navigation, Footer Links Engaged, Link: {{getElementText}}">
+      <footer data-matomo-seen="Navigation, Footer Links Seen, Tab: {{getActiveTabName}}" data-matomo-click="Navigation, Footer Links Engaged, Link: {{getElementText}}">
         <%= render partial: "layouts/libraries_footer" %>
         <%= render partial: "layouts/institute_footer" %>
       </footer>
diff --git a/app/views/search/_form.html.erb b/app/views/search/_form.html.erb
index 83ae14b0..42e33ac1 100644
--- a/app/views/search/_form.html.erb
+++ b/app/views/search/_form.html.erb
@@ -8,7 +8,7 @@
     <input id="tab-to-target" type="hidden" name="tab" value="<%= @active_tab %>">
     <button type="submit" class="btn button-primary">Search</button>
   </div>
-  <div class="search-actions"  data-matomo-click="Search, Advanced Search Engaged">
+  <div class="search-actions"  data-matomo-click="Search, Advanced Search Engaged, Tab: {{getActiveTabName}}">
     <a href="https://libraries.mit.edu/search-advanced">Advanced search</a>
   </div>
 </form>
@@ -16,22 +16,24 @@
 <% if Feature.enabled?(:boolean_picker) %>
   <aside class="panel panel-info">
     <div class="panel-heading">
-  <% if cookies[:boolean_type].present? %>
-    <p>Your boolean preference for this session is: <%= cookies[:boolean_type] %></p>
-  <% else %>
-    <p>No preference is set. Default boolean preference (`AND`) is enabled.</p>
-  <% end %>
+      <p>
+        <% if cookies[:boolean_type].present? %>
+          Your boolean preference for this session is: <%= cookies[:boolean_type] %>
+        <% else %>
+          No preference is set. Default boolean preference (`AND`) is enabled.
+        <% end %>
+      </p>
     </div>
     <div class="panel-body">
       <p>Change to:</p>
       <ul>
-      <% ENV.fetch('BOOLEAN_OPTIONS', 'AND,OR').split(',').each do |opt| %>
-        <li><%= link_to(opt, boolpref_path(boolean_type: opt)) %></li>
-      <% end %>
+        <% ENV.fetch('BOOLEAN_OPTIONS', 'AND,OR').split(',').each do |opt| %>
+          <li><%= link_to(opt, boolpref_path(boolean_type: opt)) %></li>
+        <% end %>
         <li><%= link_to('Clear preference', boolpref_path()) %></li>
       </ul>
     </div>
-    </aside>
+  </aside>
 <% end %>
 
 <%= render partial: 'search/source_tabs' if params[:q].present? %>
diff --git a/app/views/search/_results_callouts.html.erb b/app/views/search/_results_callouts.html.erb
index b092d8d3..9790d758 100644
--- a/app/views/search/_results_callouts.html.erb
+++ b/app/views/search/_results_callouts.html.erb
@@ -1,4 +1,4 @@
-<div class="callout-wrapper" data-track-content data-content-name="Search Suggestions" data-matomo-seen="Search Suggestions, Search Suggestions Seen" data-matomo-click="Search Suggestions, Search Suggestions Engaged, Link: {{getElementText}}">
+<div class="callout-wrapper" data-track-content data-content-name="Search Suggestions" data-matomo-seen="Search Suggestions, Search Suggestions Seen, Tab: {{getActiveTabName}}" data-matomo-click="Search Suggestions, Search Suggestions Engaged, Link: {{getElementText}}">
   <% if ['cdi', 'alma', 'all'].include?(@active_tab) %>
     <%= render partial: "results_callout_component", locals: {
       fa_name: 'magnifying-glass',
diff --git a/app/views/search/_results_sidebar.html.erb b/app/views/search/_results_sidebar.html.erb
index 3733406a..d4522645 100644
--- a/app/views/search/_results_sidebar.html.erb
+++ b/app/views/search/_results_sidebar.html.erb
@@ -1,6 +1,6 @@
 <aside id="results-sidebar">
   <%= render partial: "results_callouts" %>
-  <div class="core-sidebar-items" data-track-content data-content-name="Sidebar Links" data-matomo-seen="Sidebar Links, Sidebar Links Seen" data-matomo-click="Sidebar Links, Sidebar Links Engaged, Link: {{getElementText}}">
+  <div class="core-sidebar-items" data-track-content data-content-name="Sidebar Links" data-matomo-seen="Sidebar Links, Sidebar Links Seen, Tab: {{getActiveTabName}}" data-matomo-click="Sidebar Links, Sidebar Links Engaged, Link: {{getElementText}}">
     <div>
       <i class="fa-light fa-message"></i>
       <div>
diff --git a/app/views/search/results.html.erb b/app/views/search/results.html.erb
index 75e06c74..b0d3f0a1 100644
--- a/app/views/search/results.html.erb
+++ b/app/views/search/results.html.erb
@@ -10,7 +10,7 @@
   <%= render(partial: 'trigger_tacos') if tacos_enabled? %>
 
   <%= turbo_frame_tag "search-results", data: { turbo_action: "advance" } do %>
-    <div id="results-section" data-matomo-seen="Search, Search Performed">
+    <div id="results-section" data-matomo-seen="Search, Search Performed, Tab: {{getActiveTabName}}">
 
       <% if @show_primo_continuation %>
 
diff --git a/app/views/tacos/analyze.html.erb b/app/views/tacos/analyze.html.erb
index 869a3155..fdb7952c 100644
--- a/app/views/tacos/analyze.html.erb
+++ b/app/views/tacos/analyze.html.erb
@@ -1,5 +1,5 @@
 <% if @suggestions.present? && !Feature.enabled?(:geodata) %>
-  <div id="hint" aria-live="polite" data-matomo-seen="Interventions, Intervention Shown" data-track-content data-content-name="Intervention">
+  <div id="hint" aria-live="polite" data-matomo-seen="Interventions, Intervention Shown, Tab: {{getActiveTabName}}" data-track-content data-content-name="Intervention">
     <aside class="mitlib-suggestion-panel">
       <div class="panel-content">
         <h2 class="panel-type">Suggested resource</h2>
diff --git a/app/views/thirdiron/libkey.html.erb b/app/views/thirdiron/libkey.html.erb
index 7f01a5f1..4cc1ca67 100644
--- a/app/views/thirdiron/libkey.html.erb
+++ b/app/views/thirdiron/libkey.html.erb
@@ -1,22 +1,22 @@
 <% if ThirdIron.enabled? && @libkey.present? %>
   <% if @libkey[:pdf_link].present? || @libkey[:html_link].present? %>
     <% if @libkey[:pdf_link].present? %>
-      <%= link_to( @libkey[:pdf_link][:text], @libkey[:pdf_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:pdf_link][:text] } ) %>
+      <%= link_to( @libkey[:pdf_link][:text], @libkey[:pdf_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen, Tab: {{getActiveTabName}}", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:pdf_link][:text] } ) %>
     <% end %>
 
     <% if @libkey[:html_link].present? %>
-      <%= link_to( @libkey[:html_link][:text], @libkey[:html_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:html_link][:text] } ) %>
+      <%= link_to( @libkey[:html_link][:text], @libkey[:html_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen, Tab: {{getActiveTabName}}", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:html_link][:text] } ) %>
     <% end %>
 
   <%# If we don't have direct links, fall back to best_integrator_link. This allows for things like Article Galaxy links %>
   <% elsif @libkey[:best_integrator_link].present? %>
-    <%= link_to( @libkey[:best_integrator_link][:text], @libkey[:best_integrator_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:best_integrator_link][:text] } ) %>
+    <%= link_to( @libkey[:best_integrator_link][:text], @libkey[:best_integrator_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, LibKey Link Seen, Tab: {{getActiveTabName}}", matomo_click: "Results, LibKey Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:best_integrator_link][:text] } ) %>
   <% end %>
 
   <%# Display browzine link if available. This should always display if we have it regardless of other links. %>
   <% if @libkey[:browzine_link].present? %>
     <div class="libkey-actions">
-      <%= link_to @libkey[:browzine_link][:text], @libkey[:browzine_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, Browzine Link Seen", matomo_click: "Results, Browzine Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:browzine_link][:text] } %>
+      <%= link_to @libkey[:browzine_link][:text], @libkey[:browzine_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, Browzine Link Seen, Tab: {{getActiveTabName}}", matomo_click: "Results, Browzine Link Engaged, Link: {{getElementText}}", content_piece: @libkey[:browzine_link][:text] } %>
     </div>
   <% end %>
 

From 4efc6712a444b4c3b7766ef84baed5972dc47bea Mon Sep 17 00:00:00 2001
From: djanelle-mit <djanelle@mit.edu>
Date: Fri, 10 Apr 2026 14:05:15 -0400
Subject: [PATCH 2/3] Fixing missed browzine link event name

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

diff --git a/app/views/thirdiron/browzine.html.erb b/app/views/thirdiron/browzine.html.erb
index 30fc74f2..94ed2589 100644
--- a/app/views/thirdiron/browzine.html.erb
+++ b/app/views/thirdiron/browzine.html.erb
@@ -1,7 +1,7 @@
 <% if ThirdIron.enabled? && @browzine.present? %>
   <% if @browzine[:browzine_link].present? %>
     <div class="libkey-actions">
-      <%= link_to @browzine[:browzine_link][:text], @browzine[:browzine_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, Browzine Link Seen", matomo_click: "Results, Browzine Link Engaged, Link: {{getElementText}}", content_piece: @browzine[:browzine_link][:text] } %>
+      <%= link_to @browzine[:browzine_link][:text], @browzine[:browzine_link][:link], class: 'button libkey-link', data: { matomo_seen: "Results, Browzine Link Seen, Tab: {{getActiveTabName}}", matomo_click: "Results, Browzine Link Engaged, Link: {{getElementText}}", content_piece: @browzine[:browzine_link][:text] } %>
     </div>
   <% end %>
 <% end %>

From 2f6bea2dcfd5b5c985a98569d1a9008f62f7fec4 Mon Sep 17 00:00:00 2001
From: djanelle-mit <djanelle@mit.edu>
Date: Fri, 10 Apr 2026 15:57:49 -0400
Subject: [PATCH 3/3] Updated documentation

---
 app/javascript/matomo_tracking.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/javascript/matomo_tracking.js b/app/javascript/matomo_tracking.js
index 5b17bf1c..c2ef7619 100644
--- a/app/javascript/matomo_tracking.js
+++ b/app/javascript/matomo_tracking.js
@@ -5,8 +5,8 @@
 //
 // CLICK TRACKING
 // Add `data-matomo-click="Category, Action, Name"` to any element to track
-// clicks as Matomo events. The Name segment is optional. We have a convention 
-// of using semicolons inside Name to divide multiple values.
+// clicks as Matomo events. We have a convention of using semicolons inside Name 
+// to divide multiple values. This is purely for consistency.
 //
 // This tracking can be placed directly on an interactive element.
 // Tracking also works if placed on a container. The script will look
@@ -16,22 +16,22 @@
 //
 // Examples:
 //   <a href="/file.pdf" data-matomo-click="Downloads, PDF Click, My Paper">Download</a>
-//   <button data-matomo-click="Search, Boolean Toggle">AND/OR</button>
+//   <button data-matomo-click="Search, Boolean Toggle, Off">AND/OR</button>
 //
 // Event delegation on `document` means this works for elements loaded
 // asynchronously (Turbo frames, content-loader, etc.) without re-binding.
 //
 // SEEN TRACKING
 // Add `data-matomo-seen="Category, Action, Name"` to any element to fire a
-// Matomo event when that element becomes visible in the viewport. The Name
-// segment is optional. We have a convention of using semicolons inside Name 
-// to divide multiple values. Each element fires at most once per page load.
+// Matomo event when that element becomes visible in the viewport. We have  
+// a convention of using semicolons inside Name to divide multiple values. 
+// Each element fires at most once per page load.
 // Works for elements present on initial page load and for elements injected
 // later by Turbo frames or async content loaders.
 //
 // Examples:
 //   <div data-matomo-seen="Impressions, Result Card, Alma">...</div>
-//   <a data-matomo-seen="Promotions, Banner Shown">...</a>
+//   <a data-matomo-seen="Promotions, Banner Shown, Banner Type">...</a>
 //
 // DYNAMIC VALUES
 // Wrap a helper name in double curly braces anywhere inside a segment to have