304408 - Rename "Sanitize" feature in docs

Status: RESOLVED FIXED

(Firefox Graveyard :: Help Documentation, defect)

Description

[Jeff Walden (gone starting June 8)]

See bug 285064.

Comment 1

[Steffen Wilberg]

Attachment: patch

Only renames the item. This patch doesn't attempt to fix bug 302607.

Comment 2

[Jeff Walden (gone starting June 8)]

Comment on attachment 192691 [details] [diff] [review]
patch

>Index: mozilla/browser/locales/en-US/chrome/help/menu_reference.xhtml

>+  <p>Clears the items which are selected in the Clear Private Data Settings

The dialog's just "Clear Private Data".

>Index: mozilla/browser/locales/en-US/chrome/help/prefs.xhtml

>-  <h3 id="sanitize">Sanitize Settings</h3>
>+  <h3 id="sanitize">Clear Private Data Settings</h3>

Let's rev the id here to better reflect reality -- "clear_private_data" should
do fine.

>+    &brandShortName;. &pref.pluralCaps; for this are in the <em>Clear Private
>+    Data...</em> dialog.</p>

There's no ellipsis in the dialog's name.

>Index: mozilla/browser/locales/en-US/chrome/help/firebird-toc.rdf

You've got a bunch of unrelated changes in here from a bug whose patch I
reviewed recently; just be careful you don't check in too much if this gets
approved prior to approval of the patch in whichever bug it is.

>-        <rdf:li> <rdf:Description ID="menu-tools-sanitize" nc:name="Sanitize &brandShortName;..." nc:link="menu_reference.xhtml#sanitize"/> </rdf:li>
>+        <rdf:li> <rdf:Description ID="menu-tools-sanitize" nc:name="Clear Private Data..." nc:link="menu_reference.xhtml#sanitize"/> </rdf:li>

If this isn't linked from the options dialog, let's update the ID here.  (It
seems only prefs-sanitize is linked, so this looks like it should probably be
safe.)	If it is, file a bug to change it and other such instances on trunk to
something meaningful.  (Note that said bug may overlap with any cleanup Mike
Beltzner might do as followup to bug 285064, so it's probably best to leave it
alone at least until that's finalized.)  We'll deal with it when we've wrapped
up the remaining bugs that we determine are needed for 1.5 (bugs with a Target
Milestone of Firefox1.1).  I'd like to keep trunk and branch synced until we've
wrapped up those bugs, and then it's open season again. :-)

Comment 3

[Steffen Wilberg]

Attachment: fix comments

I didn't change "prefs-sanitize" since that's linked from
http://lxr.mozilla.org/seamonkey/source/browser/components/preferences/sanitize.xul#55.

Comment 4

[Steffen Wilberg]

Jeff, can you check this in for me, please? My linux system died, and I can't
install a new one because my harddisk is dying, SMART status bad :(
Bug 304333 and bug 298484 as well, please.

Comment 5

[Jeff Walden (gone starting June 8)]

(In reply to comment #4)
> Jeff, can you check this in for me, please?

Gotcha, although it may not get done until Sunday (tomorrow's the other
possibility).

Comment 6

[Jeff Walden (gone starting June 8)]

Patch checked in on branch and trunk; I opened bug 305228 to change the name of
the help topic that's referenced in prefs UI.