Closed
Bug 525035
Opened 16 years ago
Closed 16 years ago
Import button on Open panel for file-based bookmarks import is not localizable
Categories
(Camino Graveyard :: Bookmarks, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: alqahira)
References
()
Details
Attachments
(1 file)
3.75 KB,
patch
|
stuart.morgan+bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
Igor reported that that "Import" button in the Open panel that you get when choosing a file for bookmarks import is not localizable. Indeed, it's not.
This patch fixes that (with a "ImportPanelButton" key to help keep us from confusing it from the "Import" button in the nib), and it also moves all of the bookmarks import/export/opening localizable strings to be adjacent to each other, rather than in two hunks at opposite ends of the file.
l10n is considering doing this string for 2.0.1 if we do bug 521137, so flagging for 2.0.1 until there's a decision.
Flags: camino2.0.1?
Attachment #408897 -
Flags: superreview?(stuart.morgan+bugzilla)
Updated•16 years ago
|
Attachment #408897 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Comment 1•16 years ago
|
||
Comment on attachment 408897 [details] [diff] [review]
Use a localizable string
>+/* Parser error loading bookmarks.xml file */
Change "xml" to "plist" while you are moving this, so it's not years out of date ;)
sr-smorgan otherwise.
Assignee | ||
Comment 2•16 years ago
|
||
Checked in on cvs trunk with "plist".
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Flags: camino2.0.2?
Flags: camino2.0.1?
Flags: camino2.0.1-
Assignee | ||
Comment 3•16 years ago
|
||
(In reply to comment #0)
> l10n is considering doing this string for 2.0.1 if we do bug 521137, so
> flagging for 2.0.1 until there's a decision.
Minusing this as a result of the minusing in bug 521137.
Flags: camino2.0.2? → camino2.0.2-
You need to log in
before you can comment on or make changes to this bug.
Description
•