Last Comment Bug 363700 - Software Installation (pref-smartupdate.xul) prefs page is designed for xpfe SM not suiterunner
: Software Installation (pref-smartupdate.xul) prefs page is designed for xpfe ...
Product: SeaMonkey
Classification: Client Software
Component: General (show other bugs)
: Trunk
: All All
-- normal with 2 votes (vote)
: ---
Assigned To: Mark Banner (:standard8)
Depends on: suiterunner
  Show dependency treegraph
Reported: 2006-12-13 04:58 PST by Mark Banner (:standard8)
Modified: 2007-05-29 11:23 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Proposed re-layout (65.00 KB, image/png)
2007-01-14 14:35 PST, Mark Banner (:standard8)
no flags Details
Patch v1 (8.90 KB, patch)
2007-01-18 14:27 PST, Mark Banner (:standard8)
no flags Details | Diff | Splinter Review
Revised screenshot (72.52 KB, image/png)
2007-01-18 14:27 PST, Mark Banner (:standard8)
no flags Details
Patch v1a (8.58 KB, patch)
2007-01-19 05:18 PST, Mark Banner (:standard8)
iann_bugzilla: review+
Details | Diff | Splinter Review
Patch v2 (9.42 KB, patch)
2007-05-11 11:08 PDT, Mark Banner (:standard8)
iann_bugzilla: review+
neil: superreview+
Details | Diff | Splinter Review
Patch for checkin (9.83 KB, patch)
2007-05-14 14:18 PDT, Mark Banner (:standard8)
standard8: review+
standard8: superreview+
Details | Diff | Splinter Review

Description User image Mark Banner (:standard8) 2006-12-13 04:58:49 PST
The Software Installation page ( is designed for setting the prefs that were to set up update notifications for SeaMonkey as an xpfe application.

With suiterunner we're intending to use the toolkit update service for the app and extensions. We need to write a prefs page for updates specific to suiterunner - or rewrite the existing one.

Note that at the moment automatic updates are disabled by default in suiterunner - I guess we'd need to talk to MoCo/MoFo to find out what we need to do to a) let automatic updates take place b) host the facilities for automatic updates.
Comment 1 User image Mark Banner (:standard8) 2007-01-07 06:30:02 PST
When the smart update page is redone, we need to also ensure we remove the following prefs from browser-prefs.js (version in suite/browser) as they apply to xpfe version only:

Comment 2 User image Mark Banner (:standard8) 2007-01-14 14:35:32 PST
Created attachment 251465 [details]
Proposed re-layout

I'd like to propose this as the rough new layout of the Software Installation pane (also known as smartupdate). Looking at the available prefs ( I think this is all we need.
Comment 3 User image 2007-01-14 15:16:28 PST
Comment on attachment 251465 [details]
Proposed re-layout

You can't duplicate those radiobutton accesskeys!

It'd be nice to get that extensions.dss.enabled pref into the debug panel ;-)
Comment 4 User image Mark Banner (:standard8) 2007-01-15 00:05:20 PST
(In reply to comment #3)
> (From update of attachment 251465 [details])
> You can't duplicate those radiobutton accesskeys!
Opps, that's a copy and paste error

> It'd be nice to get that extensions.dss.enabled pref into the debug panel ;-)

Yep, should be easy to do.

Comment 5 User image Robert Kaiser 2007-01-15 05:20:50 PST
A button to directly launch Add-Ons Manager would probably be a good idea to fit in there.
Comment 6 User image Mark Banner (:standard8) 2007-01-18 14:27:14 PST
Created attachment 251966 [details] [diff] [review]
Patch v1

First version of the patch. Application options are disabled in the xul (with note) until we sort out arrangements for software update (probably sometime after we get suiterunner going on trunk).

The extension options work and set prefs as they should - I've also checked that an extension update (in this case it was chatzilla) is detected and I was notified on next start up (providing I hadn't gone into add-on manager which would have indicated it anyway).
Comment 7 User image Mark Banner (:standard8) 2007-01-18 14:27:58 PST
Created attachment 251967 [details]
Revised screenshot
Comment 8 User image Mark Banner (:standard8) 2007-01-19 05:18:54 PST
Created attachment 252046 [details] [diff] [review]
Patch v1a

This patch just removes a change that shouldn't have been there.
Comment 9 User image Ian Neal 2007-01-21 08:44:31 PST
Comment on attachment 252046 [details] [diff] [review]
Patch v1a

>Index: suite/locales/en-US/chrome/common/pref/pref-smartupdate.dtd
>@@ -6,6 +6,21 @@
> <!ENTITY allowedSites.label                   "Allowed Sites">
> <!ENTITY allowedSites.accesskey               "S">
>+<!-- These entities are only used in suiterunner (MOZ_XUL_APP=1 set) -->
>+<!ENTITY autoUpdates.label                    "Automatically check for updates to:">
>+<!ENTITY appUpdates.label                     "&brandShortName;">
>+<!ENTITY appUpdates.accesskey                 "y">
>+<!ENTITY appDaily.accesskey                   "d">
>+<!ENTITY appWeekly.accesskey                  "w">
>+<!ENTITY extensionsUpdates.label              "Installed Add-ons">
>+<!ENTITY extensionsUpdates.accesskey          "n">
>+<!ENTITY extensionsDaily.accesskey            "i">
>+<!ENTITY extensionsWeekly.accesskey           "k">
>+<!ENTITY appUpdatesDisabled.desc              "Automatic updates to &brandShortName; are not currently available. This feature is intended for implementation before the next release.">
>+<!ENTITY addonManagerButton.label             "Add-on Manager">
>+<!ENTITY addonManagerButton.accesskey         "g">
>+<!-- These entities are only used in xpfe SeaMonkey (MOZ_XUL_APP not set) -->
> <!ENTITY enableNotification.label             "Notify me when new and updated software is available">
> <!ENTITY enableNotification.accesskey         "N">
> <!ENTITY requireConfirmation.label            "Require manual confirmation of each install">
>@@ -18,7 +33,11 @@
> <!ENTITY updateNotifications.desc             "Check for new versions of this software.  No information about you or your computer will be revealed during this operation.">
> <!ENTITY enableUN.label                       "Check for updates:">
> <!ENTITY enableUN.accesskey                   "C">
>-<!ENTITY weekly.label                         "weekly">
>-<!ENTITY weekly.accesskey                     "w">
> <!ENTITY monthly.label                        "monthly">
> <!ENTITY monthly.accesskey                    "m">
>+<!-- These entites are used for both types of SeaMonkey -->
>+<!ENTITY daily.label                          "daily">
>+<!ENTITY daily.accesskey                      "d">
>+<!ENTITY weekly.label                         "weekly">
>+<!ENTITY weekly.accesskey                     "w">
You do not use daily.accesskey at all, weekly.accesskey is only used in xpfe SM and daily.label is only used in suiterunner. So you need to re-arrange/remove to make things correct.
r=me with those changes.
Comment 10 User image Mark Banner (:standard8) 2007-05-11 11:08:14 PDT
Created attachment 264498 [details] [diff] [review]
Patch v2

This patch is similar to v1a bug I'm planning on applying it at the time of the change over on trunk, and therefore I've removed all the ifdefs and sorted out the locale file.
Comment 11 User image 2007-05-12 11:54:47 PDT
Comment on attachment 264498 [details] [diff] [review]
Patch v2

>+          <!-- disabled until SM updates are available -->
>+          <radiogroup id="appUpdateFrequency" orient="horizontal" 
>+                      prefstring="app.update.interval"
>+                      prefdefval="86400">
>+            <radio id="appFreqDaily" label="&daily.label;" disabled="true"
>+                   accesskey="&appDaily.accesskey;" value="86400"/>
>+            <radio id="appFreqWeekly" label="&weekly.label;" disabled="true"
>+                   accesskey="&appWeekly.accesskey;" value="604800"/>
>+          </radiogroup>
I believe that these days you can put disabled="true" on the radiogroup instead of having to put it on both the radios individually.
Comment 12 User image Ian Neal 2007-05-14 13:41:45 PDT
Comment on attachment 264498 [details] [diff] [review]
Patch v2

>Index: suite/locales/en-US/chrome/common/pref/pref-smartupdate.dtd
I would look at using "e" as the accesskey for "Allow web sites..." which would free up "A" to be used below.
> <!ENTITY allowedSites.label                   "Allowed Sites">
> <!ENTITY allowedSites.accesskey               "S">
Possibly change to "A"

>+<!ENTITY autoUpdates.label                    "Automatically check for updates to:">
>+<!ENTITY appUpdates.label                     "&brandShortName;">
>+<!ENTITY appUpdates.accesskey                 "y">
Possibly change to "S" using letters like "y" or "g" is not good.

>+<!ENTITY addonManagerButton.label             "Add-on Manager">
>+<!ENTITY addonManagerButton.accesskey         "g">
Possibly change to "M" for same reason as above.

r=me with those changes discussed.
Comment 13 User image Mark Banner (:standard8) 2007-05-14 14:18:14 PDT
Created attachment 264796 [details] [diff] [review]
Patch for checkin

Addresses Ian's comments. Ready for checkin at switchover to suiterunner (when bug 328887 is resolved).
Comment 14 User image Mark Banner (:standard8) 2007-05-15 00:51:46 PDT
Moving this bug from blocking bug 328887 to depending on it as we're now not going to check this in until that bug is implemented.
Comment 15 User image Mark Banner (:standard8) 2007-05-29 11:23:23 PDT
Patch checked in -> fixed.

Note You need to log in before you can comment on or make changes to this bug.