History

Use Yes/No labels for desktop worktree confirm

Julius Marminge committed 9 months ago

Commit 10571fa1 parent 44506ba2 files · +3 −3Browse files

Unified · Split

2 files
  1. apps/desktop/src/confirmDialog.test.ts +2 −2
  2. apps/desktop/src/confirmDialog.ts +1 −1
apps/desktop/src/confirmDialog.test.ts+2−2
@@ -35,7 +35,7 @@ describe("showDesktopConfirmDialog", () => { · 34 unmodified lines
3535 expect(showMessageBoxMock).toHaveBeenCalledWith(
3636 ownerWindow,
3737 expect.objectContaining({
38− buttons: ["Cancel", "Confirm"],
38+ buttons: ["No", "Yes"],
3939 message: "Delete worktree?",
4040 }),
4141 );
@@ -49,7 +49,7 @@ describe("showDesktopConfirmDialog", () => { · 7 unmodified lines
4949 expect(result).toBe(false);
5050 expect(showMessageBoxMock).toHaveBeenCalledWith(
5151 expect.objectContaining({
52− buttons: ["Cancel", "Confirm"],
52+ buttons: ["No", "Yes"],
5353 message: "Delete worktree?",
5454 }),
5555 );
apps/desktop/src/confirmDialog.ts+1−1
@@ -13,7 +13,7 @@ export async function showDesktopConfirmDialog( · 12 unmodified lines
1313
1414 const options = {
1515 type: "question" as const,
16− buttons: ["Cancel", "Confirm"],
16+ buttons: ["No", "Yes"],
1717 defaultId: CONFIRM_BUTTON_INDEX,
1818 cancelId: 0,
1919 noLink: true,