Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#919 closed patch (fixed)

French translations updated, new fr_CH keyboard layout

Reported by: Jean-Michel <je at n-michel dot net> Owned by: danielk
Priority: minor Milestone: 0.19
Component: mythtv Version: head
Severity: medium Keywords: i18n french virtual_keyboard
Cc: Ticket locked: no

Description

The attached patch updates source:trunk/mythtv/i18n/mythfrontend_fr.ts and the menus with new french translations, up-to-date with r8469.

A new virtual keyboard layout for the swiss french keyboard layout has been added, as well as a "Français (suisse)" language setting. To support the latter, a new GetLanguageAndVariant? method has been added to MythContext.

This method returns both the language and the variant (e.g. "fr_CH" or "en_GB"), which was actually the prior behavior of the MythContext::GetLanguage?() method. This latter method now only returns the language part, i.e. the first two characters (as was documented).

The ThemedMenu? and XMLParse classes have been modified to take advantage of this; they now try to use "lang" attributes for text and values first with the language variant and then with only the language.

Attachments (3)

i18n_fr_svn8469.patch (195.2 KB) - added by Jean-Michel <je at n-michel.net> 14 years ago.
Proposed patch (see description in ticket)
i18n_fr_svn8469.patch.1 (195.2 KB) - added by Jean-Michel <je at n-michel.net> 14 years ago.
"LiveTV" was translated one time too many, or so it seems: new patch
rest_of_919.diff (6.1 KB) - added by Oscar Carlsson 14 years ago.
The rest of the diff..

Download all attachments as: .zip

Change History (10)

Changed 14 years ago by Jean-Michel <je at n-michel.net>

Attachment: i18n_fr_svn8469.patch added

Proposed patch (see description in ticket)

Changed 14 years ago by Jean-Michel <je at n-michel.net>

Attachment: i18n_fr_svn8469.patch.1 added

"LiveTV" was translated one time too many, or so it seems: new patch

comment:1 Changed 14 years ago by Oscar Carlsson

-      <text lang="CA">TV directe</text>

Why's that in there? Please doublecheck your patches before you send them in!

Changed 14 years ago by Oscar Carlsson

Attachment: rest_of_919.diff added

The rest of the diff..

comment:2 Changed 14 years ago by Oscar Carlsson

All translations applied in [8487].

comment:3 Changed 14 years ago by Oscar Carlsson

(In [8490]) Fran?\195?\167ais (suisse) virtual keyboard from Jean-Michel. Refs #919.

comment:4 Changed 14 years ago by Oscar Carlsson

Milestone: 0.19

comment:5 Changed 14 years ago by Oscar Carlsson

Owner: changed from Oscar Carlsson to danielk

Re-assigning it to you Daniel so that you don't forget it :)

comment:6 Changed 14 years ago by danielk

Resolution: fixed
Status: newclosed

(In [8620]) Closes #919, by applying patch.

The intent of the patch is to make it easier to translate variants of a language. If MythTV can't find a translation in pt_BR, it can pick up the pt_PT string instead. MythContext now returns the language and it's variant when you call GetLanguageAndVariant?() and it returns just the language when you call GetLanguage?(). I looked over the changed code and grep'ed for other uses of GetLanguage?(), but if anyone sees a translation from a different variant than expected anywhere please reopen #919 with an error report.

comment:7 Changed 13 years ago by danielk

(In [8791]) References #919.

Fixes a bug discovered by Oscar where the alternate text in a menu would be preferred over the primary text.

Note: See TracTickets for help on using tickets.