refactor: remove manual click tracking logic from project and blog components

Author: actions-user

js/blog.js

@@ -9,7 +9,7 @@ import { formatDate, formatDateShort } from './utils.js';
 import { trackEvent } from './analytics.js';
 
 async function init() {
-    const params   = new URLSearchParams(window.location.search);
+    const params = new URLSearchParams(window.location.search);
     const postPath = params.get('post');
 
     if (postPath) {
@@ -36,11 +36,6 @@ async function initListingView() {
     try {
         const manifest = await fetchCached(config.blog_manifest);
         renderListing(manifest.posts || [], document.getElementById('posts-list'));
-        // Track post clicks from listing
-        document.getElementById('posts-list')?.addEventListener('click', e => {
-            const row = e.target.closest('[data-track]');
-            if (row) trackEvent('post_read', row.dataset.track);
-        });
     } catch (err) {
         console.error(err);
         document.getElementById('posts-list').innerHTML =
@@ -72,8 +67,7 @@ function renderListing(posts, container) {
             <div class="group-label">— ${cat} —</div>
             <div class="project-group-items">
                 ${items.map(p => `
-                    <a class="blog-row" href="/writing?post=${encodeURIComponent(p.path)}"
-                       data-track="${p.title}">
+                    <a class="blog-row" href="/writing?post=${encodeURIComponent(p.path)}">
                         <span class="blog-title">${p.title}</span>
                         <span class="blog-meta">
                             <span class="blog-cat">[ ${cat} ]</span>
@@ -101,11 +95,11 @@ async function initPostView(postPath) {
             fetchTextCached(config.blog_base_url + postPath)
         ]);
 
-        const posts      = manifest.posts || [];
-        const meta       = posts.find(p => p.path === postPath);
-        const idx        = posts.findIndex(p => p.path === postPath);
-        const prevPost   = idx < posts.length - 1 ? posts[idx + 1] : null;
-        const nextPost   = idx > 0                ? posts[idx - 1] : null;
+        const posts = manifest.posts || [];
+        const meta = posts.find(p => p.path === postPath);
+        const idx = posts.findIndex(p => p.path === postPath);
+        const prevPost = idx < posts.length - 1 ? posts[idx + 1] : null;
+        const nextPost = idx > 0 ? posts[idx - 1] : null;
 
         document.title = meta ? `jebin2 — ${meta.title}` : 'jebin2 — writing';
         if (meta?.title) trackEvent('post_read', meta.title);
@@ -118,7 +112,7 @@ async function initPostView(postPath) {
 }
 
 function renderPost(mdText, meta, prevPost, nextPost, container) {
-    const cat     = meta ? getCategory(meta.path) : '';
+    const cat = meta ? getCategory(meta.path) : '';
     // Rewrite relative links/images to absolute raw GitHub URLs
     const baseUrl = meta
         ? `https://raw.githubusercontent.com/jebin2/blog/main/${getPostDir(meta.path)}/`
@@ -144,13 +138,13 @@ function renderPost(mdText, meta, prevPost, nextPost, container) {
         <nav class="post-nav">
             <div>
                 ${prevPost
-                    ? `<a href="?post=${encodeURIComponent(prevPost.path)}">← ${prevPost.title}</a>`
-                    : ''}
+            ? `<a href="?post=${encodeURIComponent(prevPost.path)}">← ${prevPost.title}</a>`
+            : ''}
             </div>
             <div>
                 ${nextPost
-                    ? `<a href="?post=${encodeURIComponent(nextPost.path)}">${nextPost.title} →</a>`
-                    : ''}
+            ? `<a href="?post=${encodeURIComponent(nextPost.path)}">${nextPost.title} →</a>`
+            : ''}
             </div>
         </nav>
     `;

js/home.js

@@ -134,18 +134,13 @@ function renderLatestPosts(manifest, container, recentCounts, allTimeCounts) {
     container.innerHTML = sorted.map((p, i) => {
         const slug = encodeURIComponent(p.path);
         return `
-            <a class="project-row" href="/writing?post=${slug}" data-track="${p.title}">
+            <a class="project-row" href="/writing?post=${slug}">
                 <span class="project-num">${String(i + 1).padStart(2, '0')}.</span>
                 <span class="project-name">${p.title}</span>
                 <span class="blog-date">${formatDateShort(p.date)}</span>
             </a>
         `;
     }).join('');
-
-    container.addEventListener('click', e => {
-        const row = e.target.closest('[data-track]');
-        if (row) trackEvent('post_read', row.dataset.track);
-    });
 }
 
 init();

js/projects.js

@@ -25,7 +25,6 @@ async function init() {
         allProjects = await fetchCached(config.projects);
         render('all');
         initFilter();
-        initTracking();
     } catch (err) {
         console.error(err);
         if (listEl) listEl.innerHTML = '<p class="state-msg">failed to load projects.</p>';
@@ -85,14 +84,6 @@ function projectRow(p) {
     `;
 }
 
-/* ---- Track project clicks ---- */
-function initTracking() {
-    document.getElementById('project-list')?.addEventListener('click', e => {
-        const row = e.target.closest('[data-track]');
-        if (row) trackEvent('project_click', row.dataset.track);
-    });
-}
-
 /* ---- Filter buttons ---- */
 function initFilter() {
     const buttons = document.querySelectorAll('.filter-btn');