fix(import): make the preview's tooltips agree with each other

They had drifted into three voices: statements of fact, an instruction,
and one that named "the file" for an import that may well have come from
a URL. Each now says what happened and, where a checkbox does something
irreversible or surprising, what checking it does — with "here" for the
workspace and "the source" for whatever was imported.

"Added since the last import" was also untrue on a first import, where
there was no last import to speak of.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Gregory Schier
2026-09-10 08:28:38 -07:00
co-authored by Claude Opus 5
parent 213458b60c
commit b22a5bbcad
@@ -660,19 +660,19 @@ function actionHelp(item: ImportPlanItem): string | null {
: text; : text;
switch (item.action) { switch (item.action) {
case "create": case "create":
return "Added since the last import"; return "Not in this workspace yet";
case "update": case "update":
return help("Changed since the last import"); return help("Changed in the source since the last import");
case "delete": case "delete":
return item.reason === "moved_into_ignored_folder" return item.reason === "moved_into_ignored_folder"
? "Moved into an ignored folder. Import that folder instead to follow the move" ? "Moved into an ignored folder. Importing that folder instead follows the move"
: "Deleted since the last import"; : "Gone from the source since the last import. Checking it deletes it here";
case "keep_local": case "keep_local":
return help("Local edits made since the last import. Importing will revert them if checked"); return help("Changed here since the last import. Checking it reverts to the source");
case "conflict": case "conflict":
return help("Changed both here and in the file since the last import"); return help("Changed here and in the source since the last import");
case "ignored": case "ignored":
return "In the file, but ignored. Check it to import it"; return "Not in this workspace. Imports leave it alone until you check it";
default: default:
return null; return null;
} }