Closed
Bug 262779
Opened 20 years ago
Closed 20 years ago
[l10n] ExportedBookmarkFile text not localizable
Categories
(Camino Graveyard :: Translations, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Camino0.9
People
(Reporter: petteri.kamppuri, Assigned: Usul)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
application/octet-stream
|
jaas
:
review+
mikepinkerton
:
superreview+
|
Details |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.2) Gecko/20040825 Camino/0.8.1
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.2) Gecko/20040825 Camino/0.8.1
Default file name, "ExportedBookmarkFile", for exporting bookmarks is not
localizable.
Reproducible: Always
Steps to Reproduce:
1. Select Export Bookmarks from the File menu
2. See what the dialog box says.
Expected Results:
It should've given me a default localized version of the file name.
Assignee | ||
Comment 1•20 years ago
|
||
Confirming. In MainController.mm
we have
"int saveResult = [savePanel runModalForDirectory:nil
file:NSLocalizedString(@"ExportedBookmarkFile", @"Exported Bookmarks")];"
there is no such entry in localized.string
Mike do you need a patch for this bug ?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•20 years ago
|
||
Also since we do bookmark stuff in 0.8.2 it would be nice if that one got included.
Assignee | ||
Comment 3•20 years ago
|
||
Assignee | ||
Updated•20 years ago
|
Attachment #161057 -
Flags: review?(me)
Assignee | ||
Updated•20 years ago
|
Attachment #161057 -
Flags: review?(joshmoz)
Comment 4•20 years ago
|
||
(In reply to comment #3)
> Created an attachment (id=161057)
> Corrected Localized strings
>
In your Localizable.strings you have:
"Exported Bookmarks" = "ExportedBookmarkFile";
Shouldn't it be:
"ExportedBookmarkFile" = "Exported Bookmarks";
Updated•20 years ago
|
Attachment #161057 -
Flags: review?(me) → review-
Assignee | ||
Updated•20 years ago
|
Attachment #161057 -
Flags: review?(joshmoz)
Assignee | ||
Comment 5•20 years ago
|
||
NeXT time I'll take my time before submiting.
Attachment #161057 -
Attachment is obsolete: true
Assignee | ||
Updated•20 years ago
|
Attachment #161175 -
Flags: review?(me)
Comment on attachment 161175 [details]
Localized.strings containg the right string
looks good to me
Attachment #161175 -
Flags: superreview?(pinkerton)
Attachment #161175 -
Flags: review?(me)
Attachment #161175 -
Flags: review+
Comment 7•20 years ago
|
||
Comment on attachment 161175 [details]
Localized.strings containg the right string
sr=pink.
go for it.
Attachment #161175 -
Flags: superreview?(pinkerton) → superreview+
landed on trunk and MOZILLA_1_7_BRANCH
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•