Closed Bug 304408 Opened 14 years ago Closed 14 years ago
Rename "Sanitize" feature in docs
See bug 285064.
Only renames the item. This patch doesn't attempt to fix bug 302607.
Assignee: nobody → steffen.wilberg
Status: NEW → ASSIGNED
Attachment #192691 - Flags: review?(jwalden+fxhelp)
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. :-)
I didn't change "prefs-sanitize" since that's linked from http://lxr.mozilla.org/seamonkey/source/browser/components/preferences/sanitize.xul#55.
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 possibility).
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.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.