Closed
Bug 564097
Opened 14 years ago
Closed 14 years ago
Move prompts.properties into commonDialog.properties
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
mozilla2.0b1
People
(Reporter: Dolske, Assigned: Dolske)
References
Details
Attachments
(1 file, 1 obsolete file)
6.33 KB,
patch
|
Gavin
:
review+
Pike
:
feedback+
|
Details | Diff | Splinter Review |
Bug 563274 is cleaning up some of the prompting code, and it's annoying to have to deal with two string bundles for a common prompt implementation. There are only 4 strings in prompts.properties, so let's just move them to a different file. Also, it's the singular "commonDialog" everywhere else (.dtd/.js/.xul), so I renamed commonDialog_s_.properties to commonDialog.properties.
Attachment #443793 -
Flags: superreview?(benjamin)
Attachment #443793 -
Flags: review?(benjamin)
Assignee | ||
Comment 1•14 years ago
|
||
(Note that bugzilla's diff-viewer is confused about the change to toolkit/locales/jar.mn; the line is being modified not deleted)
Updated•14 years ago
|
Attachment #443793 -
Flags: superreview?(benjamin)
Attachment #443793 -
Flags: superreview+
Attachment #443793 -
Flags: review?(l10n)
Attachment #443793 -
Flags: review?(benjamin)
Comment 2•14 years ago
|
||
Comment on attachment 443793 [details] [diff] [review] Patch v.1 This is an r+/-, I like the general concept of the patch, but we shouldn't do the file rename, IMHO. There's perhaps a handful of localizers that can read and use the patch as help, and will do an hg mv on their side, the others quite likely loose their work. Given that quite a few localizations don't even work on central, we can't help them either. Thus, I'd say there's too little to win with the rename here. Noting this as an r-, as quite a bit of the patch is the actual rename.
Attachment #443793 -
Flags: review?(l10n) → review-
Assignee | ||
Comment 3•14 years ago
|
||
Pike: How about if I write a script to to the hg rename in the L10N repos? And, I suppose, also automate the appending of prompts.properties to commonDialog.properties? That should eliminate any need for localizers to do any work as a result of this patch.
Comment 4•14 years ago
|
||
Yeah, doing scripts would help, if it'd be easy when to run them. For those actively working on l10n-central, we might create a race, we'll need to reach out a lot to make sure people pull and merge, and then there's a long tail for which l10n-central is totally irrelevant, as we'll branch 1.9.3 off of 1.9.2 for them. At which point we'd have to run the scripts again etc. Loads of paperwork, really.
Assignee | ||
Comment 5•14 years ago
|
||
Axel: I'm not sure if you're saying we can do this or not. I'd still like to make this change (w/rename script). I don't understand why you raise the issues in comment 4 -- mozilla-central users have similar issues every day, so I wonder why we're using a DVCS for l10n-central if we can't make use of even basic capabilities like this? The priorities here seem backward.
Comment 6•14 years ago
|
||
We have roughly 40 localization teams for which l10n-central doesn't make any sense, and won't be used as a base for fx 4. We also have a large userbase on web-based translation systems like narro or pootle, and local systems like the translation toolkit, which just export to a directory structure, with little to none support for things like renames. We should not do the rename.
Assignee | ||
Comment 7•14 years ago
|
||
Updated patch, just rolls prompts.properties into commonDialogs.properties. [Does not rename commonDialogs to commonDialog]
Attachment #443793 -
Attachment is obsolete: true
Attachment #451397 -
Flags: review?
Attachment #451397 -
Flags: feedback?(l10n)
Assignee | ||
Updated•14 years ago
|
Attachment #451397 -
Flags: review? → review?(gavin.sharp)
Updated•14 years ago
|
Attachment #451397 -
Flags: review?(gavin.sharp) → review+
Comment 8•14 years ago
|
||
Comment on attachment 451397 [details] [diff] [review] Patch v.2 looks good, yeah.
Attachment #451397 -
Flags: feedback?(l10n) → feedback+
Assignee | ||
Comment 9•14 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/9e30f0174588
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a6
You need to log in
before you can comment on or make changes to this bug.
Description
•