564097 - Move prompts.properties into commonDialog.properties

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Justin Dolske [:Dolske]]

Attachment: Patch v.1

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.

Comment 1

[Justin Dolske [:Dolske]]

(Note that bugzilla's diff-viewer is confused about the change to toolkit/locales/jar.mn; the line is being modified not deleted)

Comment 2

[Axel Hecht [:Pike]]

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.

Comment 3

[Justin Dolske [:Dolske]]

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

[Axel Hecht [:Pike]]

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.

Comment 5

[Justin Dolske [:Dolske]]

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

[Axel Hecht [:Pike]]

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.

Comment 7

[Justin Dolske [:Dolske]]

Attachment: Patch v.2

Updated patch, just rolls prompts.properties into commonDialogs.properties. [Does not rename commonDialogs to commonDialog]

Comment 8

[Axel Hecht [:Pike]]

Comment on attachment 451397 [details] [diff] [review]
Patch v.2

looks good, yeah.

Comment 9

[Justin Dolske [:Dolske]]

Pushed http://hg.mozilla.org/mozilla-central/rev/9e30f0174588