[fr] Printed messages have a double colon after some email headers in French Thunderbird

VERIFIED FIXED

Status

Mozilla Localizations
fr / French
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: MI, Assigned: Cédric Corazza)

Tracking

({verified1.8.1.15})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.14) Gecko/20080404 Firefox/2.0.0.14
Build Identifier: Thunderbird version 2.0.0.12 (20080213)

When printing an email with the French version, some (but not all) email headers have 2 colons instead of 1. For example:

  De : sender@example.com
  Pour :: someone@example.com
  Copie à :: other@example.com



Reproducible: Always

Steps to Reproduce:
1. Use Print preview with an email to see the problem




The problem seems to be in the file
 /chrome/fr.jar : /locale/fr/messenger/mime.properties

The following patch appears to solve the problem:

--- mime.properties	Tue Apr 24 11:52:00 2007
+++ mime.properties.new	Mon Apr 21 19:43:53 2008
@@ -111 +111 @@
-1010=R\u00e9pondre \u00e0\u00a0:
+1010=R\u00e9pondre \u00e0
@@ -121 +121 @@
-1012=Pour\u00a0:
+1012=Pour
@@ -127 +127 @@
-1013=Copie \u00e0\u00a0:
+1013=Copie \u00e0
@@ -132 +132 @@
-1014=Forum\u00a0:
+1014=Forum
@@ -137 +137 @@
-1015=Faire suivre \u00e0\u00a0:
+1015=Faire suivre \u00e0
@@ -177 +177 @@
-1023=Copie cach\u00e9e \u00e0\u00a0:
+1023=Copie cach\u00e9e \u00e0
(Assignee)

Updated

9 years ago
Assignee: bugzilla.fr → cedric.corazza
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 1

9 years ago
While the colon should go, I think the nbsp should stay there, and be added to some other lines.

I'm not sure it is the right way to do BTW, hardcoding a colon in the code instead of putting it in the localizable strings is sure to cause l10y problems. What happens with RTL languages?
(Reporter)

Comment 2

9 years ago
I understand your point about RTL languages. However, I suggest fixing the French double-colon first by removing it, so that this bug doesn't stay around for years. 

Then a different bug could be filed to move the colon out of the code into the localized strings in all languages.
(Assignee)

Comment 3

9 years ago
Created attachment 317296 [details] [diff] [review]
Patch removing extra semi-colons and adding nbsp where necessary
Attachment #317296 - Flags: review?(benoit.leseul)

Updated

9 years ago
Attachment #317296 - Flags: review?(benoit.leseul) → review+
(Assignee)

Updated

9 years ago
Attachment #317296 - Flags: approval1.8.1.15?
(Assignee)

Comment 4

9 years ago
FIXED on trunk
Summary: Printed messages have a double colon after some email headers in French Thunderbird → [fr] Printed messages have a double colon after some email headers in French Thunderbird

Comment 5

9 years ago
Comment on attachment 317296 [details] [diff] [review]
Patch removing extra semi-colons and adding nbsp where necessary

>Index: mime.properties
<...>
> # CC
> ## @name MIME_MHTML_CC
> ## @loc 
> # LOCALIZATION NOTE (1013): Do not translate "CC" below.
>-1013=Copie \u00e0\u00a0:
>+1013=Copie \u00e0\u00a0

Really?
(Assignee)

Comment 6

9 years ago
(In reply to comment #5)
> > # CC
> > ## @name MIME_MHTML_CC
> > ## @loc 
> > # LOCALIZATION NOTE (1013): Do not translate "CC" below.
> >-1013=Copie \u00e0\u00a0:
> >+1013=Copie \u00e0\u00a0
> 
> Really?
I'm not sure to understand. Is it about the "CC" that shouldn't be translated?
It has been for years.
Otherwise, this removes the extra colon and a non breakable space should be added before the colon added automatically as can be seen in the example in the initial description.

Comment 7

9 years ago
Comment on attachment 317296 [details] [diff] [review]
Patch removing extra semi-colons and adding nbsp where necessary

a=me, yeah, it won't get worse. Please land and test by the 6th (code freeze), even though I don't know if we're going to do a tb .15. Use the fixed... and verified1.8.1.15 keywords as usual.
Attachment #317296 - Flags: approval1.8.1.15? → approval1.8.1.15+
(Assignee)

Comment 8

9 years ago
FIXED
Checking in mail/chrome/messenger/mime.properties;
/l10n/l10n/fr/mail/chrome/messenger/mime.properties,v  <--  mime.properties
new revision: 1.2.2.2; previous revision: 1.2.2.1
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: fixed1.8.1.15
Resolution: --- → FIXED
(Assignee)

Comment 9

9 years ago
Verified.
version 2.0.0.15pre (20080605) build id: 2008060503
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1.15 → verified1.8.1.15
You need to log in before you can comment on or make changes to this bug.