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)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alqahira, Assigned: alqahira)

Details

(Whiteboard: l10n)

Attachments

(1 file, 1 obsolete file)

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.
Attached patch Probable fix (obsolete) — Splinter Review
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)
Attachment #383606 - Flags: superreview?(stuart.morgan+bugzilla) → superreview-
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.
Attached patch Probable fix, v2Splinter Review
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 on attachment 385270 [details] [diff] [review] Probable fix, v2 sr=smorgan
Attachment #385270 - Flags: superreview?(stuart.morgan+bugzilla) → superreview+
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.

Attachment

General

Created:
Updated:
Size: