Comfy-Org / Comfy-Org/Comfy-Desktop

snapshots: clicking copy-event source should navigate to that install

Open
#802 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
458
Forks
59
Avg merge
22h 18m
Merged PRs (30d)
45

Description

Follow-up to #781 / #801.

## Context

The Snapshots tab timeline renders "copy events" (when this install was created as a copy of another install) as a row showing the source's name and the copy timestamp. The source name is currently plain text:

https://github.com/Comfy-Org/ComfyUI-Desktop-2.0-Beta/blob/main/src/renderer/src/views/comfyUISettings/SnapshotsView.vue#L976-L986

## Proposal

Make the `{source}` portion of the label a hyperlink that navigates to the source install's panel (same target the picker uses when you select that install). When the source install no longer exists, fall back to plain text (or a disabled chip) so users see the historical name but don't get a dead click.

## Things to consider / scope

- **Existence check** - `CopyEvent` only carries `installationId` + `installationName` + `copiedAt`. The renderer needs to know whether an install with that id still exists before deciding link vs. plain text. Either fetch the install list it already has access to, or extend the IPC payload with a boolean.
- **Navigation target** - pick the matching existing primitive (panel switch IPC, picker selection, etc.) so we don't invent a new route.
- **Missing source UX** - disabled link with a tooltip ("Install no longer exists") vs. plain text vs. struck-through name - needs a quick design call.
- **A11y** - the label currently sits inside a `` group with an icon + time; the link should be the inner name only, with a clear accessible name.

Splitting this out of #781 so that PR stays a minimal locale fix.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.