Closed
Bug 498708
Opened 16 years ago
Closed 16 years ago
Open URL in Camino service name not localizable
Categories
(Camino Graveyard :: OS Integration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: alqahira)
Details
(Whiteboard: l10n)
Attachments
(1 file, 1 obsolete file)
13.89 KB,
patch
|
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
I noticed while looking at the Chinese 1.6.x that other apps' Services displayed in the menu with localized names, while ours doesn't.
We should just need to add that string to InfoPlist.strings for 2.0.
Assignee | ||
Comment 1•16 years ago
|
||
This should be the right fix (it's certainly what Safari does); unfortunately, I've been unable to test because I haven't figured out the right magic incantation to rebuild the Services menu using the current Camino and I have far too many copies of Camino running around :-(
Attachment #383606 -
Flags: superreview?(stuart.morgan+bugzilla)
Assignee | ||
Updated•16 years ago
|
Whiteboard: l10n
Updated•16 years ago
|
Attachment #383606 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview-
Comment 2•16 years ago
|
||
Comment on attachment 383606 [details] [diff] [review]
Probable fix
According to
http://developer.apple.com/documentation/Cocoa/Conceptual/SysServices/Concepts/properties.html
we should be using a file call ServicesMenu.strings for this. Safari actually has that as well, and that's what a bunch of other Apple apps I spot-checked do.
Assignee | ||
Comment 3•16 years ago
|
||
Switches to ServicesMenu.strings and hooks that file up to the strings-generation stuff (and adds both the .in and the generated/ file to the project).
I still can't trigger this; I don't know if the Services menu is generated at startup and cached (in addition to my having too many Caminos) or what :-(
Attachment #383606 -
Attachment is obsolete: true
Attachment #385270 -
Flags: superreview?(stuart.morgan+bugzilla)
Comment 4•16 years ago
|
||
Comment on attachment 385270 [details] [diff] [review]
Probable fix, v2
sr=smorgan
Attachment #385270 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Assignee | ||
Comment 5•16 years ago
|
||
Landed on cvs trunk.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•