Closed Bug 675781 Opened 9 years ago Closed 8 years ago
First Last does not have a comma
Attachment #549949 - Flags: review?(mbanner)
Comment on attachment 549949 [details] [diff] [review] patch I'm not really happy with how "First Last" reads, but I agree that the comma shouldn't be there. So, unasked-for-ui-r=me. ;) Later, Blake.
Attachment #549949 - Flags: ui-review+
Comment on attachment 549949 [details] [diff] [review] patch Whilst we're not changing the context, as we seem to agree that without the comma makes sense, then we also want to make sure locales get updated. At 55 locales with the comma and 12 locales without, I think it would be worth updating the string id (and syncing the .accesskey) rather than filing lots of bugs. I think we should also have an l10n comment explaining why we're not including the comma even though it is sort-of-natural to have it.
Attachment #549949 - Flags: review?(mbanner) → review-
Did you miss adding a file to the diff?
Attachment #562075 - Flags: review?(mbanner) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in before you can comment on or make changes to this bug.