Closed
Bug 401556
Opened 18 years ago
Closed 18 years ago
Ambiguous label in Prefs > Advanced > Network when clearing the cache
Categories
(Firefox :: Settings UI, defect)
Firefox
Settings UI
Tracking
()
VERIFIED
FIXED
Firefox 3 beta2
People
(Reporter: LpSolit, Assigned: benoit.leseul)
Details
(Whiteboard: [good first bug])
Attachments
(2 files)
|
23.48 KB,
image/png
|
Details | |
|
1.62 KB,
patch
|
Gavin
:
review+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
In Prefs > Private, you have a button labelled "Clear data now", and the same label is displayed in Prefs > Advanced > Network about clearing the cache. In the latter case, this label is ambiguous as it makes me think it's going to clear all my data, not only the cache. I suggest we rename the label to "Clear cache" or something like that. Unless it really clears everything too?? See, I'm not even sure what this button does.
Comment 1•18 years ago
|
||
Yeah, these use the same localized string. The French localizers used a more specific string for the button text ("Clear my tracks now") than en-US ("Clear Now"). I suppose this could be fixed by changing the french locale, but it would probably be a good idea to use separate entities here.
Updated•18 years ago
|
| Reporter | ||
Updated•18 years ago
|
Summary: Ambiguous label in Prefs > Advanded > Network when clearing the cache → Ambiguous label in Prefs > Advanced > Network when clearing the cache
There is no good way to fix the French locale as long as the same entity is used in both cases. To make a long story short, in French "Clear" is not really the same as "Empty" (it depends on the subject, you can clear your tracks but not a container like the cache), and both are transitive: Clear /what/? How does one clear a "Now"?.
The current label is the best compromize that could be found with only one entity, as Privacy preferences are more likely to be used than Advanced preferences.
Let's try this.
Attachment #287222 -
Flags: review?(gavin.sharp)
Comment 4•18 years ago
|
||
Comment on attachment 287222 [details] [diff] [review]
Use a separate entity in the Advanced pref pane
Merci! :)
Attachment #287222 -
Flags: review?(gavin.sharp) → review+
Updated•18 years ago
|
Attachment #287222 -
Flags: approval1.9?
Updated•18 years ago
|
Assignee: nobody → benoit.leseul
Keywords: helpwanted
Updated•18 years ago
|
Attachment #287222 -
Flags: approval1.9? → approval1.9+
Updated•18 years ago
|
Keywords: checkin-needed
Comment 5•18 years ago
|
||
Checking in browser/components/preferences/advanced.xul;
/cvsroot/mozilla/browser/components/preferences/advanced.xul,v <-- advanced.xul
new revision: 1.41; previous revision: 1.40
done
Checking in browser/locales/en-US/chrome/browser/preferences/advanced.dtd;
/cvsroot/mozilla/browser/locales/en-US/chrome/browser/preferences/advanced.dtd,v <-- advanced.dtd
new revision: 1.26; previous revision: 1.25
done
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 M10
Comment 6•18 years ago
|
||
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b2pre) Gecko/2007112704 Minefield/3.0b2pre
for clear cache button used &clearCacheNow.label;
Verified Fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•