Incorrectly translated %s variables

Until someone corrects them, I've demoted the translations to suggestions in Launchpad

It looks like the %s, %s is translated in a strange way in Hebrew
https://translations.edge.launchpad.net/ubuntu/karmic/+source/language-selector/+pots/language-selector/he/10/+translate

Maybe I'm missing something

It may be the directionality that makes it look that way.

Someone seems to have accepted the suggestions without change, and I believe they are still wrong. To the Hebrew team: do you think you could review them again?

Looking at other translated PO files in Hebrew, it seems to confirm that variables must be translated as they are, that is, %s.

It looks like Rosetta interface is handling RTL strings. I will download the PO file and take a look at the original text.

Meanwhile I have assigned this problem to ubuntu-l10n-he as they can help with this matter.

Cheers

This string is corect, just that Rosseta is displaying it in RTL

This is the PO file

#: ../LanguageSelector/gtk/GtkLanguageSelector.py:418
#, python-format
msgid "%s, %s"
msgstr "%s, %s"

These are exactly the same string as mentioned by Adi, I can add an RLM or
LRM mark if needed

Where does this text appear and what do the %s stand for?

Yaron, unfortunalty there is no comment associated with the string (this can be another bug).

So currently it should be as is (the current string is copied from the
original one)
If anyone knows what should appear instead of these variables I can make the
string more accurate using the RLM and LRM unicode chars respectfully

The missing translation comment is:

"Translators: this string will concatenate the "%n to install" and "%n to remove" strings, you can replace the comma if you need to."

The comment was in the wrong location, I have moved it to the correct spot.

This bug was fixed in the package language-selector - 0.5.3

---------------
language-selector (0.5.3) lucid; urgency=low

  [ Michael Vogt ]
  * when checking for is_admin check for uid == 0 too

  [ Arne Goetje ]
  * GtkLanguageSelector.py: move a Translators comment to the right
    location, so that it ends up in the.pot file. (LP: #456290)
  * Added fallback for Frisian (fy_NL and fy_DE) to data/languagelist.
    (LP: #537540)
  * check if the dependency package for translations is available in the
    cache (LP: #527891)
  * restored --show-installed option in check-language-support
  * fix language-selector.desktop.in (Thanks to Stéphane Loeuillet)
    (LP: #146973)
  * fix a crash when language-selector is run by a non-admin user
 -- Arne Goetje <email address hidden> Fri, 19 Mar 2010 17:54:18 +0800