Closed Bug 301748 Opened 19 years ago Closed 18 years ago

calIFileType isn't l10n friendly

Categories

(Calendar :: Internal Components, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mvl, Assigned: jminta)

Details

Attachments

(1 file)

the calIFileType interface (defined in calIImportExport.idl) isn't l10n frienly.
It has to return strings, while it should return stringbundle references or
something like that. But it still should be pluggable.
Unfortunately, I do not have cycles to work on Calendar stuff these days (just as it's getting to the good part!), so I am a bad owner for these bugs.  To delete the tragically-large chunk of bugspam, search for gregorianabdication.
Assignee: shaver → nobody
Attached patch use stringbundles — — Splinter Review
More l10n stuff that I've like to get into 0.3a2.  Moves the importers and exporters to using stringbundles.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #216815 - Flags: first-review?(mvl)
Comment on attachment 216815 [details] [diff] [review]
use stringbundles

I assume you will also actually add the strings to the .properties file? :)
r=mvl with that fixed.
Attachment #216815 - Flags: first-review?(mvl) → first-review+
Patch checked in.  The strings in the .properties file were in my local tree.  not sure why they didn't make it into the patch, but they're checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: