Opened 21 years ago

Closed 21 years ago

Last modified 21 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> 21 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> 21 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 21 years ago.
The rest of the diff..

Download all attachments as: .zip

Change History (10)

by Jean-Michel <je at n-michel.net>, 21 years ago

Attachment: i18n_fr_svn8469.patch added

Proposed patch (see description in ticket)

by Jean-Michel <je at n-michel.net>, 21 years ago

Attachment: i18n_fr_svn8469.patch.1 added

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

comment:1 by Oscar Carlsson, 21 years ago

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

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

by Oscar Carlsson, 21 years ago

Attachment: rest_of_919.diff added

The rest of the diff..

comment:2 by Oscar Carlsson, 21 years ago

All translations applied in [8487].

comment:3 by Oscar Carlsson, 21 years ago

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

comment:4 by Oscar Carlsson, 21 years ago

Milestone: 0.19

comment:5 by Oscar Carlsson, 21 years ago

Owner: changed from Oscar Carlsson to danielk

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

comment:6 by danielk, 21 years ago

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 by danielk, 21 years ago

(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.