A few translation problems

Lapinoo

Joined: 2004-05-08
Posts: 378

Posted: Thu, 2006-01-19 11:49

While updating the fr.po files, I noticed a few things that require attention :

Module rearrange
In "Change the order of the items in this album.%s Click an item to move, then click the new location.", the "%s" is used to insert a , which is not XHTML compliant.
This string should be cut in two strings.

Module comments
The module is displayed as "Comments" in the left column in the Admin page, whichever is the activated language.

Module hidden
The good old trick to factorize action is used : "Hide %s"... As previously discussed, it only works in english ("Hide photo", "Hide album"...).
This should be replaced by "Hide photo", "Hide album", and so on as it was done for "Edit photo" instead of "Edit %s"...

I haven't finished to update fr.po files, but I still suspect there are more glitches to fix...

valiant

Joined: 2003-01-04
Posts: 32509

Posted: Thu, 2006-01-19 12:03

> In "Change the order of the items in this album.%s Click an item to move, then click the new location.", the "%s" is used to insert a , which is not XHTML compliant. This string should be cut in two strings.

1. we insert a not a . it's in a paragraph. why is it not XHTML compliant? is a valid parent class. http://learningforlife.fsu.edu/webmaster/references/xhtml/tags/text/br.cfm#parent_child
2. keeping a single long string makes sense since it makes the context more obvious.

> The module is displayed as "Comments" in the left column in the Admin page, whichever is the activated language.

thanks. fixed in cvs.

> The good old trick to factorize action is used : "Hide %s"... As previously discussed, it only works in english ("Hide photo", "Hide album"...).
> This should be replaced by "Hide photo", "Hide album", and so on as it was done for "Edit photo" instead of "Edit %s"...

can you add a link to the previous discussion. this works in German too... but i see that the translations achieved with this method are probably not very precise / adequate.

Lapinoo

Joined: 2004-05-08
Posts: 378

Posted: Thu, 2006-01-19 13:33

1 - Yup... In fact I used "View source" within FireFox, I know remember that the source is slightly modified before being displayed.

I understand keeping a long sentence might ease the understanding, however, when translating, I wondered "WTF is this %s for?"... So I tested, and looked for this string once installed to view what the %s was remplaced with. It was not obvious at first sight that a was going to be inserted... I thought that another piece of sentence was going to be inserted.

Also, I remember that the po files are not sorted anymore. If you cut this string in two pieces, then they should be together in the po file.
As far as I am concerned, it is easier for me to translate short sentences that longer ones with %s used for non obvious reasons.

3 - Check this topic:
http://gallery.menalto.com/node/33890

stephenju

Joined: 2005-07-05
Posts: 172

Posted: Thu, 2006-01-19 23:08

There are similar broken up sentences regarding the new Reposiory feature in Core module. Like "As of the last update on" which I assume will be appended with some date time string. And ", the repository contains..." which I think will be appended to something else. This is assuming the translators know how the programmer was thinking and happen to use the same grammar as English again.

Also, there are some Unix shell commands in the localization file, too. Like "$ cd %s" and "$ chmod 777 %s". I am not sure those command should ever be translated. If not, they should be removed from the translation files.

Most if not all of the new modules do not have any po files in the CVS yet. I created some of them myself but don't know if the package name and version are correct. It also errors when I try to create po file for Picasa.

mindless

Joined: 2004-01-04
Posts: 8601

Posted: Fri, 2006-01-20 01:20

3) hidden module now supports specific translations for "Hide Album", "Hide Photo" and "Hide Movie"

mindless

Joined: 2004-01-04
Posts: 8601

Posted: Fri, 2006-01-20 02:07

stephenju, broken up sentence fixed, unix commands un-marked for translation, some other cleanup in repository code.
Fixed picasa strings and added makefile for picasa/po directory.

for new po files just put:
Gallery: {module name} 1.0
for Project-Id-Version.. we don't track the actual version number in the po files, just 1.0

stephenju

Joined: 2005-07-05
Posts: 172

Posted: Fri, 2006-01-20 04:50

mindless, thanks for the quick fix. I just finished and uploaded translation based on CVS from 2 days ago. Will update and add the new items once the update is commited to CVS.

When will the test repository go live so we can check out the translation? It's kind of like shooting in the dark without even seeing the English text in live pages.

mindless

Joined: 2004-01-04
Posts: 8601

Posted: Fri, 2006-01-20 16:45

unfortunately downloadable plugins didn't make the cut for 2.1, so don't worry about those strings.. they'll remain inactive until 2.2

adding recent translation submissions now....

stephenju

Joined: 2005-07-05
Posts: 172

Posted: Fri, 2006-01-20 18:45

Quote:

unfortunately downloadable plugins didn't make the cut for 2.1, so don't worry about those strings..

Now you tell me! I've finished all the translation already!

stephenju

Joined: 2005-07-05
Posts: 172

Posted: Sat, 2006-01-21 01:01

I am stuck with CVS update:

cvs update: [01:00:16] waiting for mindless's lock in 
/cvsroot/gallery/gallery2/modules/publishxp/locale/it/LC_MESSAGES

mindless, are you still commiting changes?

stephenju

Joined: 2005-07-05
Posts: 172

Posted: Sat, 2006-01-21 03:39

Never mind. It's OK now.

Lapinoo

Joined: 2004-05-08
Posts: 378

Posted: Sun, 2006-01-22 13:27

I've found a few more untranslatable strings:
In the "Edit Album" page, "Album" tab:

Order: "Apply to all subalbums"
Theme: " Use this theme in all subalbums"
Thumbanils: "Use this thumbnail size in all subalbums"
Resized pictures: "Use these target sizes in all subalbums"
Size Limit: "Use these size limits in all subalbums"
Rating: "... and for all subalbums"

mindless

Joined: 2004-01-04
Posts: 8601

Posted: Sun, 2006-01-22 17:38

Good catch, thanks. Fixed in cvs.

Lapinoo Joined: 2004-05-08 Posts: 378	Posted: Thu, 2006-01-19 11:49
	While updating the fr.po files, I noticed a few things that require attention : Module rearrange In "Change the order of the items in this album.%s Click an item to move, then click the new location.", the "%s" is used to insert a <br>, which is not XHTML compliant. This string should be cut in two strings. Module comments The module is displayed as "Comments" in the left column in the Admin page, whichever is the activated language. Module hidden The good old trick to factorize action is used : "Hide %s"... As previously discussed, it only works in english ("Hide photo", "Hide album"...). This should be replaced by "Hide photo", "Hide album", and so on as it was done for "Edit photo" instead of "Edit %s"... I haven't finished to update fr.po files, but I still suspect there are more glitches to fix...
	XUser login Username: * Password: * Create new account Request new password
valiant Joined: 2003-01-04 Posts: 32509	Posted: Thu, 2006-01-19 12:03
	> In "Change the order of the items in this album.%s Click an item to move, then click the new location.", the "%s" is used to insert a <br>, which is not XHTML compliant. This string should be cut in two strings. 1. we insert a <br/> not a <br>. it's in a <p> paragraph. why is it not XHTML compliant? <p> is a valid parent class. http://learningforlife.fsu.edu/webmaster/references/xhtml/tags/text/br.cfm#parent_child 2. keeping a single long string makes sense since it makes the context more obvious. > The module is displayed as "Comments" in the left column in the Admin page, whichever is the activated language. thanks. fixed in cvs. > The good old trick to factorize action is used : "Hide %s"... As previously discussed, it only works in english ("Hide photo", "Hide album"...). > This should be replaced by "Hide photo", "Hide album", and so on as it was done for "Edit photo" instead of "Edit %s"... can you add a link to the previous discussion. this works in German too... but i see that the translations achieved with this method are probably not very precise / adequate.

Lapinoo Joined: 2004-05-08 Posts: 378	Posted: Thu, 2006-01-19 13:33
	1 - Yup... In fact I used "View source" within FireFox, I know remember that the source is slightly modified before being displayed. I understand keeping a long sentence might ease the understanding, however, when translating, I wondered "WTF is this %s for?"... So I tested, and looked for this string once installed to view what the %s was remplaced with. It was not obvious at first sight that a <br /> was going to be inserted... I thought that another piece of sentence was going to be inserted. Also, I remember that the po files are not sorted anymore. If you cut this string in two pieces, then they should be together in the po file. As far as I am concerned, it is easier for me to translate short sentences that longer ones with %s used for non obvious reasons. 3 - Check this topic: http://gallery.menalto.com/node/33890

stephenju Joined: 2005-07-05 Posts: 172	Posted: Thu, 2006-01-19 23:08
	There are similar broken up sentences regarding the new Reposiory feature in Core module. Like "As of the last update on" which I assume will be appended with some date time string. And ", the repository contains..." which I think will be appended to something else. This is assuming the translators know how the programmer was thinking and happen to use the same grammar as English again. Also, there are some Unix shell commands in the localization file, too. Like "$ cd %s" and "$ chmod 777 %s". I am not sure those command should ever be translated. If not, they should be removed from the translation files. Most if not all of the new modules do not have any po files in the CVS yet. I created some of them myself but don't know if the package name and version are correct. It also errors when I try to create po file for Picasa.

mindless Joined: 2004-01-04 Posts: 8601	Posted: Fri, 2006-01-20 01:20
	3) hidden module now supports specific translations for "Hide Album", "Hide Photo" and "Hide Movie"

mindless Joined: 2004-01-04 Posts: 8601	Posted: Fri, 2006-01-20 02:07
	stephenju, broken up sentence fixed, unix commands un-marked for translation, some other cleanup in repository code. Fixed picasa strings and added makefile for picasa/po directory. for new po files just put: Gallery: {module name} 1.0 for Project-Id-Version.. we don't track the actual version number in the po files, just 1.0

stephenju Joined: 2005-07-05 Posts: 172	Posted: Fri, 2006-01-20 04:50
	mindless, thanks for the quick fix. I just finished and uploaded translation based on CVS from 2 days ago. Will update and add the new items once the update is commited to CVS. When will the test repository go live so we can check out the translation? It's kind of like shooting in the dark without even seeing the English text in live pages.

mindless Joined: 2004-01-04 Posts: 8601	Posted: Fri, 2006-01-20 16:45
	unfortunately downloadable plugins didn't make the cut for 2.1, so don't worry about those strings.. they'll remain inactive until 2.2 adding recent translation submissions now....

stephenju Joined: 2005-07-05 Posts: 172	Posted: Fri, 2006-01-20 18:45
	Quote: unfortunately downloadable plugins didn't make the cut for 2.1, so don't worry about those strings.. Now you tell me! I've finished all the translation already!

stephenju Joined: 2005-07-05 Posts: 172	Posted: Sat, 2006-01-21 01:01
	I am stuck with CVS update: cvs update: [01:00:16] waiting for mindless's lock in /cvsroot/gallery/gallery2/modules/publishxp/locale/it/LC_MESSAGES mindless, are you still commiting changes?

stephenju Joined: 2005-07-05 Posts: 172	Posted: Sat, 2006-01-21 03:39
	Never mind. It's OK now.

Lapinoo Joined: 2004-05-08 Posts: 378	Posted: Sun, 2006-01-22 13:27
	I've found a few more untranslatable strings: In the "Edit Album" page, "Album" tab: Order: "Apply to all subalbums" Theme: " Use this theme in all subalbums" Thumbanils: "Use this thumbnail size in all subalbums" Resized pictures: "Use these target sizes in all subalbums" Size Limit: "Use these size limits in all subalbums" Rating: "... and for all subalbums"

mindless Joined: 2004-01-04 Posts: 8601	Posted: Sun, 2006-01-22 17:38
	Good catch, thanks. Fixed in cvs.

A few translation problems

XUser login