Closed Bug 671921 Opened 10 years ago Closed 10 years ago

Minor string change: "Double check" in New Account Wizard should probably be hyphenated

Categories

(Thunderbird :: Account Manager, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 8.0

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(1 file)

This string (which appears after performing STR in bug 671917)...
>  double_check_email=Double check this email address!
...probably wants to say "Double-check this email address!"
(needs a hyphen between "Double" and "check")

Link to source:
http://mxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/messenger/accountCreation.properties#67

Justification:
==============
* http://dictionary.reference.com/browse/double+check says that the non-hyphenated version refers to a particular move in chess, whereas the hyphenated version has the meaning that we clearly intend.

* http://www.kcitraining.com/styleguide/hyphens.html says "Two or more words that act together to form a new meaning are hyphenated" and gives double-check as an example. (no idea how authoritative kcitraining.com is, but their explanation seems to make sense)
Blake can you ui-r? this idea ?
Whiteboard: [has l10n impact]
Not without a patch…  But yeah, I would buy a hyphen between double and check.
Ok, here's a patch. :)

(Let me know if there are any other reviews/approvals I should request on this)
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #547964 - Flags: ui-review?(bwinton)
OS: Linux → All
Hardware: x86_64 → All
Version: unspecified → Trunk
Comment on attachment 547964 [details] [diff] [review]
fix: s/Double check/Double-check/

Review of attachment 547964 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, ui-r=me, and r=me if you can get a localizer to comment that we don't need to change the name of the string.

(r- if we do need to change the name of the string for localization purposes, of course.  ;)
Attachment #547964 - Flags: ui-review?(bwinton)
Attachment #547964 - Flags: ui-review+
Attachment #547964 - Flags: review+
Ok! I don't see why this would require a string-name-change. (but I don't know much about l10n)

Perhaps Axel can confirm my assumption (or suggest someone who could)?
(In reply to comment #4)
> Looks good, ui-r=me, and r=me if you can get a localizer to comment that we
> don't need to change the name of the string.

You're fixing a typo in your language and that doesn't change the meaning of the string, so I don't see any need to change the entity's name.
What flod said. Good as is.
Thanks! Landed: http://hg.mozilla.org/comm-central/rev/fb61d037609a

(In reply to comment #6)
> You're fixing a typo in your language and that doesn't change the meaning of
> the string, so I don't see any need to change the entity's name.

Does that mean the "[has l10n impact]" whiteboard status can be cleared?
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Depends on what you use it for, I guess.
(In reply to Daniel Holbert [:dholbert] from comment #8)
> Thanks! Landed: http://hg.mozilla.org/comm-central/rev/fb61d037609a
> 
> (In reply to comment #6)
> > You're fixing a typo in your language and that doesn't change the meaning of
> > the string, so I don't see any need to change the entity's name.
> 
> Does that mean the "[has l10n impact]" whiteboard status can be cleared?

Yes
Whiteboard: [has l10n impact]
Target Milestone: --- → Thunderbird 8.0
You need to log in before you can comment on or make changes to this bug.