Bootstrapper

installer/bootstrap-theme.wxl

Index: installer/bootstrap-theme.wxl
===================================================================
new file mode 100644
--- /dev/null
+++ b/installer/bootstrap-theme.wxl
@@ -0,0 +1,50 @@
+<?xml version="1.0" encoding="utf-8"?>
+<WixLocalization Culture="en-us" Language="1033" xmlns="http://schemas.microsoft.com/wix/2006/localization">
+  <String Id="BundleName">Adblock Plus for IE</String>
+  <String Id="Caption">[WixBundleName] Setup</String>
+  <String Id="Title">[WixBundleName]</String>
+  <String Id="ConfirmCancelMessage">Are you sure you want to cancel?</String>

[Wladimir Palant, 2013/08/01 10:30:33]

I think this is the more common way to phrase it:

Are you sure you wish to abort installation?

[Eric, 2013/08/01 18:58:37]

The problem is that there's only one cancellation dialog no matter what you're
cancelling: (1) initial installation dialog or (2) repair/uninstall dialog. It's
a defect, IMO, that these use the same string. But fixing it means rewriting the
whole UI code.

I left it generic so that it basically fits both situations.

+  <String Id="HelpHeader">Setup Help</String>
+  <String Id="HelpText">/install | /repair | /uninstall | /layout [directory] - installs, repairs, uninstalls or
+   creates a complete local copy of the bundle in directory. Install is the default.
+
+/passive | /quiet -  displays minimal UI with no prompts or displays no UI and
+   no prompts. By default UI and all prompts are displayed.
+
+/norestart   - suppress any attempts to restart. By default UI will prompt before restart.
+/log log.txt - logs to a specific file. By default a log file is created in %TEMP%.</String>
+  <String Id="HelpCloseButton">&amp;Close</String>
+  
+  <!-- Install dialog -->
+  <String Id="InstallHeader">Setup Start</String>
+  <String Id="InstallWelcome">This setup program installs Adblock Plus for IE into your "Program Files" folder in directory "Adblock Plus for IE".</String>

[Felix Dahlke, 2013/08/01 09:51:21]

I think this sounds a bit awkward, can't we have something like: This setup
program installs Adblock Plus for IE to "%PROGRAM_FILES%/Adblock Plus for IE" Or
maybe just: This setup program installs Adblock Plus for IE to your "Program
Files" directory.

[Wladimir Palant, 2013/08/01 10:30:33]

I don't think we should refer to environment variables here, our users won't
know about them. So I would go with your second suggestion:

This setup program installs [WixBundleName] to your "Program Files" directory.

Note that we should use WixBundleName consistently.

[Felix Dahlke, 2013/08/01 10:52:43]

Well, I meant to actually show the full path to the target folder,
%PROGRAM_FILES% isn't even a variable. But yeah, might be better to go with the
simpler second variant.

[Eric, 2013/08/01 18:58:37]

The biggest problem with this message is that there's no way (not that I could
find) to give a different message based on a 32-bit or 64-bit platform. So a
full technical disclosure is impossible with the current tool.

Furthermore, I'm not ever sure that %...% substitution is even supported. The
only reason I even know about the [...] substitution is that it was already used
in the example code. There's no documentation about the syntax.

[Eric, 2013/08/01 18:58:37]

I'll do that. It also appears again below.

+  <String Id="InstallInstallButton">&amp;Install</String>
+  <String Id="InstallCloseButton">&amp;Do not install</String>

[Wladimir Palant, 2013/08/01 10:30:33]

The user doesn't choose between installing and not installing - that choice has
been made already, Adblock Plus should be installed. So the second button should
have a generic name, "Cancel" is the usual one IMHO. Also, I think that it
should be called that on all screens (meaning no separate ProgressCancelButton
string).

[Eric, 2013/08/01 18:58:37]

"Cancel" it is.
Sorry, can't change the string names. These are names hard-wired in the C# UI
code.

+
+  <!-- Progress dialog -->
+  <String Id="ProgressHeader">Setup Progress</String>
+  <String Id="ProgressLabel">Processing:</String>
+  <String Id="OverallProgressPackageText">Initializing...</String>
+  <String Id="ProgressCancelButton">&amp;Cancel install</String>
+
+  <!-- Modify dialog -->  
+  <String Id="ModifyHeader">Modify Setup</String>
+  <String Id="ModifyNotice">Adblock Plus for IE is already installed on this machine. If it's not working correctly, you may repair it. You may also uninstall it.</String>
+  <String Id="ModifyRepairButton">&amp;Repair</String>
+  <String Id="ModifyUninstallButton">&amp;Uninstall</String>
+  <String Id="ModifyCloseButton">&amp;Do nothing</String>

[Wladimir Palant, 2013/08/01 10:30:33]

Again, I would rather have a generic "Cancel" button here.

+  
+  <!-- Success dialog -->
+  <String Id="SuccessHeader">Setup Successful</String>
+  <String Id="SuccessLaunchButton">&amp;Launch</String>

[Wladimir Palant, 2013/08/01 10:30:33]

Launch what exactly? This should probably say: "Start Internet Explorer"

[Eric, 2013/08/01 18:58:37]

This is the generic text. It's not invoked at present. I left the definition
here in case we use it. If we end up needing it, then what you've suggested is
probably the right text.

+  <String Id="SuccessRestartText">You must restart your computer before you can use the software.</String>
+  <String Id="SuccessRestartButton">&amp;Restart</String>
+  <String Id="SuccessCloseButton">&amp;Done</String>

[Wladimir Palant, 2013/08/01 10:30:33]

If we are asking the user to restart then the options should read "Restart Now"
and "Restart Later" IMHO.

[Eric, 2013/08/01 18:58:37]

Again, these are not used.

+  
+  <!-- Failure dialog -->
+  <String Id="FailureHeader">Setup Failed</String>
+  <String Id="FailureHyperlinkLogText">One or more issues caused the setup to fail. Please fix the issues and then retry setup. For more information see the &lt;a href="#"&gt;log file&lt;/a&gt;.</String>
+  <String Id="FailureRestartText">You must restart your computer to complete the rollback of the software.</String>
+  <String Id="FailureRestartButton">&amp;Restart</String>
+  <String Id="FailureCloseButton">&amp;Done</String>

[Wladimir Palant, 2013/08/01 10:30:33]

Same here, "Restart Now" and "Restart Later" in case of restart IMHO.

+</WixLocalization>