I’m aware that l10n can be a nuisance for a lot of developers – and some localizers (e.g. me) can be a real pain in the *** – but when reviewing a patch that involve a change of existing strings you only have a short and quick checklist to follow:
- Does the patch fix a typo or does it make a substantial change to the string? In the first case just fixing the string is fine, in the latter case you need to change the string ID, since not all localization tools (or localizers who simply use a text editor) can catch this kind of change.
- Are you changing a string ID? Always check if there’s an associated access key and maintain the relation STRINGID.label <-> STRINGID.accesskey (again, localization tools rely on this kind of structure).
Once in a while a mistake can happen, but three times in a few days seems a bit out of average 😉
3 responses to “Dear reviewer,”
Super useful post! We started a page here to document these requirements. Can you make sure everything there is right?
https://developer.mozilla.org/en/Making_String_Changes
Hi Dietrich, thanks! I made some small changes, hope they’re ok 😉
I’m gratfuel you made the post. It’s cleared the air for me.