Closed Bug 304408 Opened 14 years ago Closed 14 years ago

Rename "Sanitize" feature in docs


(Firefox Graveyard :: Help Documentation, defect)

Not set


(Not tracked)



(Reporter: jwalden+fxhelp, Assigned: steffen.wilberg)



(Keywords: fixed1.8, late-l10n)


(1 file, 1 obsolete file)

9.66 KB, patch
: review+
Details | Diff | Splinter Review
Attached patch patch (obsolete) — Splinter Review
Only renames the item. This patch doesn't attempt to fix bug 302607.
Assignee: nobody → steffen.wilberg
Attachment #192691 - Flags: review?(jwalden+fxhelp)
Comment on attachment 192691 [details] [diff] [review]

>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. :-)
Attachment #192691 - Flags: review?(jwalden+fxhelp)
Attachment #192691 - Flags: review+
Attachment #192691 - Flags: approval1.8b4?
Attached patch fix commentsSplinter Review
I didn't change "prefs-sanitize" since that's linked from
Attachment #192691 - Attachment is obsolete: true
Attachment #192725 - Flags: review+
Attachment #192725 - Flags: approval1.8b4?
Attachment #192691 - Flags: approval1.8b4?
Target Milestone: --- → Firefox1.1
Attachment #192725 - Flags: approval1.8b4? → approval1.8b4+
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.
Whiteboard: [checkin needed]
(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
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.
Closed: 14 years ago
Resolution: --- → FIXED
Keywords: fixed1.8, late-l10n
Whiteboard: [checkin needed]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.