From 6c378426edfe161da7e42162d7c99102f2c053a4 Mon Sep 17 00:00:00 2001 From: "bugzilla%standard8.demon.co.uk" Date: Thu, 24 Aug 2006 17:34:43 +0000 Subject: [PATCH] Bug 349438 (bug 266945 enhanced / followup) add title and set default button to no for the remove all password manager confirm prompt. p=Serge Gautherie ,r=mconnor --- .../passwordmgr/resources/content/passwordManager.js | 7 +++---- .../en-US/chrome/passwordmgr/passwordmgr.properties | 1 + 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/toolkit/components/passwordmgr/resources/content/passwordManager.js b/toolkit/components/passwordmgr/resources/content/passwordManager.js index d21f70bb548..36fa5d2ad28 100644 --- a/toolkit/components/passwordmgr/resources/content/passwordManager.js +++ b/toolkit/components/passwordmgr/resources/content/passwordManager.js @@ -169,11 +169,10 @@ function DeleteAllSignons() { // Confirm the user wants to remove all passwords var dummy = { value: false }; if (prompter.confirmEx(window, - null, + kSignonBundle.getString("removeAllPasswordsTitle"), kSignonBundle.getString("removeAllPasswordsPrompt"), - prompter.BUTTON_TITLE_YES * prompter.BUTTON_POS_0 + - prompter.BUTTON_TITLE_NO * prompter.BUTTON_POS_1, - null, null, null, null, dummy) == 1) //1 = "No" button + prompter.STD_YES_NO_BUTTONS + prompter.BUTTON_POS_1_DEFAULT, + null, null, null, null, dummy) == 1) // 1 == "No" button return; DeleteAllFromTree(signonsTree, signonsTreeView, diff --git a/toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties b/toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties index 5008a5d71ee..0e847edc339 100644 --- a/toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties +++ b/toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties @@ -49,3 +49,4 @@ hidePasswords=Hide Passwords showPasswords=Show Passwords noMasterPasswordPrompt=Are you sure you wish to show your passwords? removeAllPasswordsPrompt=Are you sure you wish to remove all passwords? +removeAllPasswordsTitle=Remove all passwords