From 24fe84664697cc30b3755693f8e90b7ac60a9083 Mon Sep 17 00:00:00 2001
From: Ethanfel <ethan.fel@ts-pc.fr>
Date: Thu, 26 Feb 2026 13:39:14 +0100
Subject: [PATCH] Skip shared ancestors in expanded timeline, switch workflow
 on picker click

Expanded timeline: non-active branch rows now only show their unique
markers after the fork point, indented to align with the active branch.
Removes the dimmed-ancestor approach in favor of cleaner layout.

Workflow picker: clicking a workflow that is open in ComfyUI now
switches to that tab instead of just viewing its snapshots.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 js/snapshot_manager.js | 35 ++++++++++++++++++++++++++++-------
 1 file changed, 28 insertions(+), 7 deletions(-)

diff --git a/js/snapshot_manager.js b/js/snapshot_manager.js
index 64b2446..c67fd49 100644
--- a/js/snapshot_manager.js
+++ b/js/snapshot_manager.js
@@ -2191,9 +2191,6 @@ const CSS = `
     background: rgba(59, 130, 246, 0.12);
     border-left-color: #3b82f6;
 }
-.snap-timeline-marker-dimmed {
-    opacity: 0.35;
-}
 .snap-diff-overlay {
     position: fixed;
     inset: 0;
@@ -2836,6 +2833,23 @@ async function buildSidebar(el) {
                 if (entry.workflowKey === currentKey) {
                     viewingWorkflowKey = null;
                 } else {
+                    // Try to switch to the workflow in ComfyUI
+                    const wfStore = app.extensionManager?.workflow;
+                    const openWfs = wfStore?.openWorkflows;
+                    if (openWfs && wfStore.openWorkflow) {
+                        const target = openWfs.find(wf =>
+                            (wf.key || wf.filename || wf.path) === entry.workflowKey
+                        );
+                        if (target) {
+                            try {
+                                collapsePicker();
+                                await wfStore.openWorkflow(target);
+                                // openWorkflow listener handles viewingWorkflowKey reset + refresh
+                                return;
+                            } catch { /* fall through to view-only */ }
+                        }
+                    }
+                    // Workflow not open in ComfyUI — view its snapshots only
                     viewingWorkflowKey = entry.workflowKey;
                 }
                 activeBranchSelections.clear();
@@ -3553,7 +3567,7 @@ function buildTimeline() {
     canvasParent.appendChild(bar);
     timelineEl = bar;
 
-    function buildMarker(rec, { dimmed = false, onClickBranch = null } = {}) {
+    function buildMarker(rec, { onClickBranch = null } = {}) {
         const marker = document.createElement("div");
         marker.className = "snap-timeline-marker";
 
@@ -3571,7 +3585,6 @@ function buildTimeline() {
             marker.classList.add("snap-timeline-marker-current");
             marker.style.setProperty("--snap-marker-color", "#10b981");
         }
-        if (dimmed) marker.classList.add("snap-timeline-marker-dimmed");
 
         let tip = `${rec.label} — ${formatTime(rec.timestamp)}\n${iconInfo.label}`;
         if (rec.notes) tip += `\n${rec.notes}`;
@@ -3630,10 +3643,13 @@ function buildTimeline() {
                 const isActiveBranch = branchLeafId === currentLeafId;
                 if (isActiveBranch) row.classList.add("snap-timeline-branch-row-active");
 
+                let skippedCount = 0;
                 for (const rec of branch) {
-                    const isSharedAncestor = !isActiveBranch && currentIds.has(rec.id);
+                    if (!isActiveBranch && currentIds.has(rec.id)) {
+                        skippedCount++;
+                        continue;
+                    }
                     const marker = buildMarker(rec, {
-                        dimmed: isSharedAncestor,
                         onClickBranch: isActiveBranch ? null : () => {
                             selectBranchContaining(branchLeafId, tree);
                         },
@@ -3641,6 +3657,11 @@ function buildTimeline() {
                     row.appendChild(marker);
                 }
 
+                // Indent non-active rows so fork points align with the active branch
+                if (skippedCount > 0) {
+                    row.style.paddingLeft = `${6 + skippedCount * 24}px`;
+                }
+
                 track.appendChild(row);
             }
             return;