[translation] Need for a comment for #35

Bug #1210341 reported by londumas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-system-settings
Fix Released
Low
Unassigned
ubuntu-system-settings (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

In the translation the string #35: "%1 %" can't be understand. Either this string doesn't need to be translated or it needs to and so translators need a comment to be able to understand the context and then translate it.

https://translations.launchpad.net/ubuntu-system-settings/trunk/+pots/ubuntu-system-settings/fr/+translate?batch=10&show=all&search=%251+%25

Related branches

Revision history for this message
Jiri Grönroos (jiri-gronroos) wrote :

It's a variable that describes current charge level, see http://bazaar.launchpad.net/~system-settings-touch/ubuntu-system-settings/trunk/view/head:/plugins/battery/PageComponent.qml#L59

Some languages have a rule that the percent sign should be right after the numeric value, like having a question mark or period at the end of sentence. But this is not true for all languages, thus the developers have set this string translatable. It's the right thing to do.

I have to agree with you that Ubuntu Touch applications' translation templates do suffer from the lack of comments. Comments on context, possible variable values, possible maximum length of translated text, all these are mostly missing now.

David Planella (dpm)
description: updated
Changed in ubuntu-system-settings:
status: New → Fix Committed
importance: Undecided → Low
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-system-settings at revision None, scheduled for release in ubuntu-system-settings, milestone Unknown

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-system-settings - 0.1+13.10.20130926.4-0ubuntu1

---------------
ubuntu-system-settings (0.1+13.10.20130926.4-0ubuntu1) saucy; urgency=low

  [ Charles Kerr ]
  * As requested, remove the linefeed between a function's return type
    and its name.

  [ Sebastien Bacher ]
  * security: hide non working PIN and app access settings.
  * battery: hide screen idle delay selection, the backend is not
    working.
  * timezone: some tweaks and fixes - hide the selector, we don't have a
    backend for automatic selection - replace the text widget by label
    from the ubuntu uitk, it fixes the issue with the text being too
    small - update the placeholder string according to the new design.
  * datetime: drop empty list item, it's not in the design.
  * hide reset options until we get some backend code working. The
    factory resets require support for the system image side, the other
    options should be doable from our side later on.
  * hide phone settings, we lack phone app support for call
    forwarding,waiting.
  * update the translation template.
  * background: revert use of the new content-hub, it's buggy (see bug
    #1231368). (LP: #1231368)

  [ William Hua ]
  * Switch to ListView as workaround to LP #1227249. (LP: #1227249)

  [ David Planella ]
  * Add a translator comment for the battery charge level (bug 1210341).
    (LP: #1210341)

  [ Alberto Mardegan ]
  * Load testability driver if the -testability option is given. This
    change is needed in order to make the system-settings application
    and its plugins testable with autopilot (or other testability
    plugins).

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 392
 -- Ubuntu daily release <email address hidden> Thu, 26 Sep 2013 21:44:53 +0000

Changed in ubuntu-system-settings (Ubuntu):
status: New → Fix Released
Changed in ubuntu-system-settings:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.