Skip to content

Prompt to kill locking processes when accepting#707

Merged
SimonCropp merged 2 commits into
mainfrom
Prompt-to-kill-locking-processes-when-accepting
Jul 5, 2026
Merged

Prompt to kill locking processes when accepting#707
SimonCropp merged 2 commits into
mainfrom
Prompt-to-kill-locking-processes-when-accepting

Conversation

@SimonCropp

Copy link
Copy Markdown
Member

When accepting a move fails because the files are locked (eg the snapshot is open in Word), show a dialog listing the locked files and locking processes instead of silently leaving the move pending. Options: Ignore, Kill and accept, Kill and accept all pending, or Always kill.

Always kill persists to settings (AlwaysKillLockingProcesses) and is toggleable in the Options dialog, so future locks are killed without prompting. Existing per-tool KillLockingProcess (Word/Excel) still auto-kills without a prompt.

Split FileLockKiller into GetLockingProcesses (names the processes for the dialog) and Kill; Tracker now re-adds a move kept pending so it can be retried.

When accepting a move fails because the files are locked (eg the snapshot
is open in Word), show a dialog listing the locked files and locking
processes instead of silently leaving the move pending. Options: Ignore,
Kill and accept, Kill and accept all pending, or Always kill.

Always kill persists to settings (AlwaysKillLockingProcesses) and is
toggleable in the Options dialog, so future locks are killed without
prompting. Existing per-tool KillLockingProcess (Word/Excel) still
auto-kills without a prompt.

Split FileLockKiller into GetLockingProcesses (names the processes for the
dialog) and Kill; Tracker now re-adds a move kept pending so it can be
retried.
@SimonCropp SimonCropp added this to the 19.3.0 milestone Jul 5, 2026
@SimonCropp SimonCropp merged commit ce2687b into main Jul 5, 2026
5 checks passed
@SimonCropp SimonCropp deleted the Prompt-to-kill-locking-processes-when-accepting branch July 5, 2026 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant