Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#12874 closed (fixed)

Updated French translation

Reported by: stephaner 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 stephaner 6 years ago.
Full French PO file (4)
fr-po.diff (89.7 KB) - added by stephaner 6 years ago.
A convenient diff for the review (4)

Download all attachments as: .zip

Change History (13)

comment:1 Changed 6 years ago by david

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement set
  • Triage Stage changed from Unreviewed to Accepted

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 6 years ago by stephaner

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 6 years ago by david

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

comment:4 Changed 6 years ago by david

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

comment:5 Changed 6 years ago by david

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 6 years ago by stephaner

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 6 years ago by stephaner

  • 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 6 years ago by david

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

Changed 6 years ago by stephaner

Full French PO file (4)

Changed 6 years ago by stephaner

A convenient diff for the review (4)

comment:9 Changed 6 years ago by stephaner

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 6 years ago by jezdez

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:11 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

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