Closed Bug 641252 Opened 9 years ago Closed 9 years ago

Update help for redesigned Software Installation preferences

Categories

(SeaMonkey :: Help Documentation, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1final

People

(Reporter: rsx11m.pub, Assigned: rsx11m.pub)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 2 obsolete files)

Bug 606482 has made some substantial changes in the Edit > Preferences >
Advanced > Software Installation pane which need to be documented, and
bug 639968 will introduce further changes.

Tasks for this bug:
 - Restructure help content in cs_nav_prefs_advanced.xhtml to match the new
   layout per bug 606482 (bug 639968 would include an update on the new pref);
 - replace old "Manage Permissions" text with reference to the Data Manager.

Per bug 639968 comment #20, help changes aren't subject to the string freeze, but nevertheless it's desired to get this updated as soon as possible.
Documenting the new data manager itself is bug 599097, thus a reference from this page to it should suffice with any specifics regarding this pane.
The description of the Add-ons Manager introduced by bug 373457 may need some update as well, specifically for the "Recommended Add-ons" section after the layout change in the "Get Add-Ons" page. This could be a separate bug if it doesn't exist yet and needs to cover a broader scope.
Depends on: 571527
Attached patch Proposed patch (obsolete) — Splinter Review
This patch mainly reshuffles things to reflect the new order of options and removes any obsolete options. I've removed the description of the old dialog
for website permissions but couldn't add any link to the Data Manager as it's still waiting to be written.

As for the nesting of the lists, I've stopped at the third level given that it starts looking weird if you'd really copy the nesting of the preferences in an exact hierarchy. I think it's still obvious how they relate to each other.

I didn't touch the Add-on Manager part, the description from bug 639968 also remains unchanged as it still applies in the new context.
Assignee: nobody → rsx11m.pub
Status: NEW → ASSIGNED
Attachment #527817 - Flags: review?(iann_bugzilla)
Attached patch Proposed patch (v2) (obsolete) — Splinter Review
Sorry, I forgot that I wanted to move the general description of the Add-on Manager upfront before describing the Add-ons group.
Attachment #527817 - Attachment is obsolete: true
Attachment #527817 - Flags: review?(iann_bugzilla)
Attachment #527820 - Flags: review?(iann_bugzilla)
Comment on attachment 527820 [details] [diff] [review]
Proposed patch (v2)

>+++ b/suite/locales/en-US/chrome/common/help/cs_nav_prefs_advanced.xhtml	Fri Apr 22 13:40:55 2011 -0500
>+  <li><strong>Add-ons</strong>:
>     <ul>
>+      <li><strong>Allow websites to install extensions and updates</strong>:
Nit: it is "add-ons" rather than "extensions" now.

>+        Select this if you want to allow websites to install extensions and
Nit: and again.

r=me with that fixed.
Attachment #527820 - Flags: review?(iann_bugzilla) → review+
Thanks Ian.

(In reply to comment #5)
> >+      <li><strong>Allow websites to install extensions and updates</strong>:
> Nit: it is "add-ons" rather than "extensions" now.

Forgot to double-check the label when copy-pasting, both occurrences fixed.

Two more minor fixes applied in this patch:
 - I've changed "updated notifications" to just "updates" as they are now
   installing automatically if so selected (thus, no longer just notify);
 - "Add-on Manager" is now actually "Add-ons Manager", I've corrected that
   and will file follow-up bugs for any UI and help changes as necessary.
Attachment #527820 - Attachment is obsolete: true
Attachment #527942 - Flags: review+
Attachment #527942 - Flags: approval-seamonkey2.1?
Push for comm-central, please. This is Help content only, thus requesting approval for comm-2.0 in parallel.

The spin-off bugs are bug 652327 and bug 652328.
Keywords: checkin-needed
Whiteboard: [c-n: comm-central]
Comment on attachment 527942 [details] [diff] [review]
Final patch [Checkin: comment 10]

comm-2.0 is open, and help is not affected by l10n freeze
Attachment #527942 - Flags: approval-seamonkey2.1? → approval-seamonkey2.1+
Whiteboard: [c-n: comm-central] → [c-n: comm-central][c-n comm-20]
(In reply to comment #6)
>  - "Add-on Manager" is now actually "Add-ons Manager", I've corrected that
>    and will file follow-up bugs for any UI and help changes as necessary.

This change is only valid for the trunk, so whoever pushes this to branch needs to change it back or require a separate patch. Alternate approach: Change it back completely and leave the actual change to bug 652328.
Comment on attachment 527942 [details] [diff] [review]
Final patch [Checkin: comment 10]

http://hg.mozilla.org/comm-central/rev/52c2ccceb1d7
http://hg.mozilla.org/releases/comm-2.0/rev/41ebfb00ca65

Same for both: Reverted to "Add-on Manager" since I think this is better to be handled in its own bug, and you didn't request review for that change.

Dropping approval since none is needed.
Attachment #527942 - Attachment description: Final patch → Final patch [Checkin: comment 10]
Attachment #527942 - Flags: approval-seamonkey2.1+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: comm-central][c-n comm-20]
Target Milestone: --- → seamonkey2.1final
> (comment #9) This change is only valid for the trunk,

Actually, for Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0.2pre) Gecko/20110423 SeaMonkey/2.1pre, both the window decoration and the tab titles show the plural for "Add-ons Manager" as well, thus I figured it's the same for both. 

> (comment #10) Reverted to "Add-on Manager" since I think this is better to be
> handled in its own bug, and you didn't request review for that change.

Agreed, I'm fine with deferring this to bug 652328 to have it in a single bug. Sorry for exceeding what's acceptable in changes after a review approval...

Thanks for getting this in!
(In reply to comment #11)
> > (comment #9) This change is only valid for the trunk,
> 
> Actually, for Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0.2pre) Gecko/20110423
> SeaMonkey/2.1pre, both the window decoration and the tab titles show the plural
> for "Add-ons Manager" as well, thus I figured it's the same for both. 

Oh, my bad then. I should have checked myself. It seems the change goes at least all the way back to bug 554007 which landed 2010-04-29. Anyway, as I said, better in its own bug.
You need to log in before you can comment on or make changes to this bug.