Closed
Bug 652327
Opened 14 years ago
Closed 14 years ago
Toolkit's Add-on Manager is now Add-ons Manager, update SeaMonkey's "Tools" menu label
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.2
People
(Reporter: rsx11m.pub, Assigned: rsx11m.pub)
References
Details
Attachments
(1 file)
2.55 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
Toolkit has changed "Add-on Manager" to "Add-ons Manager" now, thus respective UI elements will need updating. I'll file a separate bug for updating the Help content as it may need further updates and wiring beyond that, also to distinguish between comm-central and comm-2.0 changes.
I expected to find more references of "Add-on Manager" that need to be updated, but apparently the only case other than Help (covered by bug 652328) is in the "Tools" menu of all windows. I'll post a patch tomorrow unless anybody is aware of any other label that needs to be changed.
Summary: Toolkit's Add-on Manager is now Add-ons Manager, adjust UI where necessary → Toolkit's Add-on Manager is now Add-ons Manager, update SeaMonkey's "Tools" menu label
Comment 2•14 years ago
|
||
FTR: This seems to at least go back all the way to bug 554007 which landed 2010-04-29. Firefox's menu entry reads "Add-ons", though.
Yes, Firefox doesn't use the "Manager" in their menu. However, SeaMonkey has "Data Manager", "Password Manager", and "Download Manager" already, thus I'd keep it "Add-ons Manager" for consistency (and it's also the tab title).
This patch contains just a single label change in tasksOverlay which applies to the Tools > Add-on(s) menu in all windows. I'm not changing any use of "addon" to "addons" in the code, that would be overdoing it given that the two terms are almost identical and no ambiguity occurs.
Following Edmund's tradition introduced in bug 606482 to use "addOnsManager", and given that entity names are case insensitive, I've changed those in this way so that localizers will pick it up (Jens, let me know if you think that this wouldn't suffice as entity change).
Attachment #528006 -
Flags: review?(neil)
Comment 5•14 years ago
|
||
Sigh. Our l10n story sucks that we can't just add an "s" without a rewrite...
Updated•14 years ago
|
Attachment #528006 -
Flags: review?(neil) → review+
Yeah, it sure would be nice if such content changes could be picked up by l10n without the need for inventing new entity names all the time.
Push for trunk please (this won't go into comm-2.0).
Keywords: checkin-needed
Whiteboard: [c-n: comm-central]
Comment 7•14 years ago
|
||
Comment on attachment 528006 [details] [diff] [review]
Patch for new label [Checkin: comment 7]
http://hg.mozilla.org/comm-central/rev/0c9c4a20b02a
Attachment #528006 -
Attachment description: Patch for new label → Patch for new label [Checkin: comment 7]
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: comm-central]
Updated•14 years ago
|
Target Milestone: --- → seamonkey2.2a1
You need to log in
before you can comment on or make changes to this bug.
Description
•