Bug 232512

Caret browser dialogue box is potentially confusing to new users
make the buttons be OK/Cancel instead of Yes/Cancel
patch by Andreas Kunz <durbacher@gmx.de> r=neil.parkwaycc.co.uk sr=alecf

xpfe/global/resources/content/bindings/browser.xml

@@ -393,10 +393,9 @@
             var buttonPressed = promptService.confirmEx(window, 
               this.mStrBundle.GetStringFromName('browsewithcaret.checkWindowTitle'), 
               this.mStrBundle.GetStringFromName('browsewithcaret.checkLabel'),
-              (promptService.BUTTON_TITLE_IS_STRING * promptService.BUTTON_POS_0) +
+              (promptService.BUTTON_TITLE_OK * promptService.BUTTON_POS_0) +
               (promptService.BUTTON_TITLE_CANCEL * promptService.BUTTON_POS_1),
-              this.mStrBundle.GetStringFromName('browsewithcaret.checkButtonLabel'),
-              null, null,
+              null, null, null,
               this.mStrBundle.GetStringFromName('browsewithcaret.checkMsg'), 
               checkValue);
             if (buttonPressed != 0)

xpfe/global/resources/locale/en-US/tabbrowser.properties

@@ -3,4 +3,3 @@ tabs.untitled=(Untitled)
 browsewithcaret.checkMsg=Do not show me this dialog box again.
 browsewithcaret.checkWindowTitle=Caret Browsing
 browsewithcaret.checkLabel=Pressing F7 turns Caret Browsing on or off. This feature places a moveable cursor in web pages, allowing you to select text with the keyboard. Do you want to turn Caret Browsing on?
-browsewithcaret.checkButtonLabel=Yes