Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#12874 closed (fixed)

Updated French translation

Reported by: Stephane Raimbault Owned by: nobody
Component: Translations Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description


Attachments (2)

django.po (129.2 KB) - added by Stephane Raimbault 7 years ago.
Full French PO file (4)
fr-po.diff (89.7 KB) - added by Stephane Raimbault 7 years ago.
A convenient diff for the review (4)

Download all attachments as: .zip

Change History (13)

comment:1 Changed 7 years ago by David Larlet

Needs documentation: unset
Needs tests: unset
Patch needs improvement: set
Triage Stage: UnreviewedAccepted

s/succés/succès/
"sélectionné(e|s)" is a bit geeky to translate "selected" but I can't find a better solution.
"All/Select all" can be translated by "Tou(te)s/Sélectionnez tou(te)s".

comment:2 Changed 7 years ago by Stephane Raimbault

I've fixed the typo on succès, thank you.
I don't want to mess the translation with gender agreement (the parenthesis for plural form are already ugly and I hope admin users will be tolerants), as said in the comments the best would be to have a gettext plural form.

Before updating the PO file, do you know how to translate the 9 missing countries?

comment:3 Changed 7 years ago by David Larlet

I wonder if we should translate strings under localflavor/* (except fr/ obviously). I will ask Jannis about that.

comment:4 Changed 7 years ago by David Larlet

Suggestion from IRC, replacing "Tous les %(total_count)s" (happens twice) by "Les %(total_count)s".

comment:5 Changed 7 years ago by David Larlet

Ok, Jannis confirms that strings marked as translations... should be translated (silly me.).

Still from IRC:

  • is the "# Here be dragon!" comment pertinent?
  • s/9~chiffres/9 chiffres/

comment:6 Changed 7 years ago by Stephane Raimbault

The comment was to catch your attention (to remove in the final version, FIXME too)! Did you see the awful associated trick (double s)?

9~chiffres, my bad, it's the way to insert unbreakable space under LaTeX not in gettext (replaced by an ASCII unbreakable space), the translation has been updated to use unbreakable spaces before or after some other digits.

The translation doesn't contain the following yet:

  • Suggestion from IRC, replacing "Tous les %(total_count)s" (happens twice) by "Les %(total_count)s".
  • 9 countries still missing

comment:7 Changed 7 years ago by Stephane Raimbault

Patch needs improvement: unset

The French translation is now complete.
I haven't changed the last translator name, feel free to change if better.

comment:8 Changed 7 years ago by David Larlet

To me, you should put your name.
Now given that it depends on Claude, you should directly ask him.

Changed 7 years ago by Stephane Raimbault

Attachment: django.po added

Full French PO file (4)

Changed 7 years ago by Stephane Raimbault

Attachment: fr-po.diff added

A convenient diff for the review (4)

comment:9 Changed 7 years ago by Stephane Raimbault

I didn't want to upload yet another PO file to just change the last translator name, but of course, that's the usual way to work on translations so now it's done.

comment:10 Changed 7 years ago by Jannis Leidel

Resolution: fixed
Status: newclosed

(In [12469]) Fixed #12874 - Updated French locale. Thanks, Stéphane Raimbault.

comment:11 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

Note: See TracTickets for help on using tickets.
Back to Top