both "email" and "e-mail" in the same sentence. Patch changes "email" -> "e-mail"

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

franz’s picture

Status: Active » Needs review

ops...

Status: Needs review » Needs work

The last submitted patch, drupa.interfacetypo.patch, failed testing.

franz’s picture

Status: Needs work » Needs review
FileSize
836 bytes

Oops, wrong patch command...

franz’s picture

Issue tags: +Novice

Adding tag

franz’s picture

Version: 7.x-dev » 8.x-dev

I think this needs to go on 8.x as well.

franz’s picture

Issue tags: -Novice

#3: drupa.interfacetypo.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Novice

The last submitted patch, drupa.interfacetypo.patch, failed testing.

franz’s picture

Title: Typo on interface text » Typo on interface text - 'emai' spelling
Status: Needs work » Needs review
FileSize
844 bytes

Re-rolling

franz’s picture

Title: Typo on interface text - 'emai' spelling » Typo on interface text - 'email' spelling
James_Stallings’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs backport to D6, +Needs backport to D7

Looks good, D8/7/6 please.

webchick’s picture

Heh. What a silly bug. I confirmed in http://drupal.org/style-guide/content that we use "e-mail" even though this isn't quite standardized in some other places from a quick grep.

I don't personally think this is worth breaking strings for in a stable release, though. I checked with Gábor and he agreed for D6.

So removing the tags and marking back to just 8.x. But it's good to go.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Comitted to 8.x. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.