CrowdIn is set up

The l10n of Pale Moon. Rawr.

Moderator: satrow

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 11:27

Moonchild wrote:It's not a directory structure problem. there is literally an issue with the zip archive itself. I don't know what you used to zip it up but it's broken.
Oops, sorry, my mistake, it was actually 7zip with zip extension) Here is true zip, please import it to Russian locale.
Here are the add-ons I made in a spare time. That was fun!

dark_moon

Re: CrowdIn is set up

Unread post by dark_moon » 2016-11-09, 11:56

Moonchild wrote:You're likely blocking too much stuff again.
The funny think is: i set nearly all settings to default, but i found the problem.
The site use websockets & HTML5 storage.

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 13:13

Russian file has been imported.

As for websockets and dom storage -- what did you expect from a full translation web application?
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

dark_moon

Re: CrowdIn is set up

Unread post by dark_moon » 2016-11-09, 13:50

I never see any site which used websocket, so i don't know.

Currently i have the german translation at 79%

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 13:54

Moonchild wrote:Russian file has been imported.
Thanks, Russian translation is 100% now! I'd like to start testing, could you upload current tree somewhere?
Here are the add-ons I made in a spare time. That was fun!

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 14:08

GMforker wrote:Is there a reason to have this files?

Code: Select all

.mkdir.done
bookmarks.html
chrome.manifest
en-US.manifest
firefox-l10n.js
install.rdf
localstore.rdf
mimeTypes.rdf
They were imported from the language pack.
I'm new to this whole system myself, so you'll have to forgive me if it's not perfectly set up in one go.
This is the kind of stuff our proposed lead person should figure out, as well as potential tools to synchronize changed source localization and/or Github integration.
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 14:14

GMforker wrote:Is there a reason to have this files?

Code: Select all

bookmarks.html
chrome.manifest
en-US.manifest
firefox-l10n.js
install.rdf
As far as I see, these ones should be localized.
Here are the add-ons I made in a spare time. That was fun!

GMforker
Astronaut
Astronaut
Posts: 537
Joined: 2015-08-27, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread post by GMforker » 2016-11-09, 14:49

Moonchild wrote:They were imported from the language pack.
I'm new to this whole system myself, so you'll have to forgive me if it's not perfectly set up in one go.
This is the kind of stuff our proposed lead person should figure out, as well as potential tools to synchronize changed source localization and/or Github integration.
Thank you, I take note of it.
JustOff wrote:
GMforker wrote:Is there a reason to have this files?

Code: Select all

bookmarks.html
chrome.manifest
en-US.manifest
firefox-l10n.js
install.rdf
As far as I see, these ones should be localized.
Yes, it has happened (if it make sense to use).

But (AFAIK):

Code: Select all

en-US.manifest
I don't know if he has the correct name ([lang].manifest?).
And the thread with the same name...

I leave it as it is.

User avatar
KNTRO
Astronaut
Astronaut
Posts: 636
Joined: 2013-01-01, 10:46
Location: ᴀᴍéʀɪᴄᴀ ʟᴀᴛɪɴᴀ
Contact:

Re: CrowdIn is set up

Unread post by KNTRO » 2016-11-09, 15:42

Off-topic:
JustOff wrote: Oops, sorry, my mistake, it was actually 7zip with zip extension) Here is true zip, please import it to Russian locale.
This is the only setting you should use when creating Zip files by using 7-Zip. Any other setting/customization under ".zip" compression with 7-Zip will potentially break it —for some reason I don't know—.
Image
”kulupu li wile e sama mute e ken mute ala la, kulupu li jo e mute ala pi ijo tu.
kulupu li wile e ken mute e sama mute ala la, kulupu li jo e mute pi ijo tu.”
jan Miton Piman

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 15:55

Off-topic:
KNTRO wrote:This is the only setting you should use when creating Zip files by using 7-Zip
I have no 7zip at all)) That was a kind of crazy usage of FAR manager)
Here are the add-ons I made in a spare time. That was fun!

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 17:01

Aand.. the first headaches are there

Problem: .properties files have all colons, equal signs, etc. escaped with a backslash
Problem: .properties files have all files prepended with an advertising line (generator)
Problem: .properties files have all extended characters \u escaped (BIG problem for e.g. ar, zh, ru, etc.)
Problem: .dtd files have all between-tags and in-tags text HTML-entity-encoded where they shouldn't be. (e.g. &brandShortName; -> &brandsShortname;)

Files are unusable as a result.
I thought they were supposed to natively support the correct formats?
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 19:07

Moonchild wrote:Files are unusable as a result.
I thought they were supposed to natively support the correct formats?
I even was sure of it, because I saw that ABP uses CrowdIn for several years. And I still don't know any free alternative of comparable convinience level.

So, I think now we have two opportunities: ask for help or write post-processor. The last one seems to me preferable. Please upload current tree of Russian localization somewhere, I'd like to play with it.
Here are the add-ons I made in a spare time. That was fun!

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 21:35

Here's the result of CrowdIn's build+save process for Russian
Attachments
ru.zip
(446.09 KiB) Downloaded 33 times
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 22:14

Moonchild wrote:Here's the result of CrowdIn's build+save process for Russian
Thank you, I'll try do deal with it.

I also sent the letter to CrowdIn support with the detailed description of the problem, let's see what they will answer.
Here are the add-ons I made in a spare time. That was fun!

GMforker
Astronaut
Astronaut
Posts: 537
Joined: 2015-08-27, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread post by GMforker » 2016-11-09, 22:17

Moonchild wrote:Aand.. the first headaches are there

Problem: .properties files have all colons, equal signs, etc. escaped with a backslash
Problem: .properties files have all files prepended with an advertising line (generator)
Problem: .properties files have all extended characters \u escaped (BIG problem for e.g. ar, zh, ru, etc.)
Problem: .dtd files have all between-tags and in-tags text HTML-entity-encoded where they shouldn't be. (e.g. &brandShortName; -> &brandsShortname;)

Files are unusable as a result.
I thought they were supposed to natively support the correct formats?
i.e.:
1.

Code: Select all

chrome/AB-CD/locale/AB-CD/global/aboutSupport.dtd
aboutSupport.userJSDescription

{[=-a id='prefs-user-js-link'-=]} instead of <a id='prefs-user-js-link'>
2.
I propose to look at:

Code: Select all

chrome/AB-CD/locale/AB-CD/global/printPreview.dtd
close.label

It should only: "Close" (translated)
Otherwise it doesn't work properly...

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 22:31

GMforker wrote:

Code: Select all

chrome/AB-CD/locale/AB-CD/global/aboutSupport.dtd
aboutSupport.userJSDescription
{[=-a id='prefs-user-js-link'-=]} instead of <a id='prefs-user-js-link'>
Thanks, that will cause two more replace patterns.

Code: Select all

chrome/AB-CD/locale/AB-CD/global/printPreview.dtd
close.label
It should only: "Close" (translated)
This looks like & -> &, already mentioned by Moonchild.
Here are the add-ons I made in a spare time. That was fun!

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 22:38

@Justoff: how about I make you a manager for the project, because I really can't sink any more time into it right now?
That way you should be able to fiddle with it on the CrowdIn site as well and export/change what is needed, and ask for assistance from their staff if/when needed.
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

GMforker
Astronaut
Astronaut
Posts: 537
Joined: 2015-08-27, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread post by GMforker » 2016-11-09, 22:39

JustOff wrote:This looks like & -> &, already mentioned by Moonchild.
It is a button to close the preview window, not closing the application.

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1759
Joined: 2015-09-03, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread post by JustOff » 2016-11-09, 22:46

Moonchild wrote:@Justoff: how about I make you a manager for the project, because I really can't sink any more time into it right now?
That way you should be able to fiddle with it on the CrowdIn site as well and export/change what is needed, and ask for assistance from their staff if/when needed.
Ok, I'll try not to break anything, but better you do make full backup of current project before)
Here are the add-ons I made in a spare time. That was fun!

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 23689
Joined: 2011-08-28, 17:27
Location: 58°2'16"N 14°58'31"E
Contact:

Re: CrowdIn is set up

Unread post by Moonchild » 2016-11-09, 23:15

JustOff wrote:Ok, I'll try not to break anything, but better you do make full backup of current project before)
You're now a manager. I'll make sure to make a fresh build, save and TM backup in the next half hour.
Improving Mozilla code: You know you're on the right track with code changes when you spend the majority of your time deleting code.

"If you want to build a better world for yourself, you have to be willing to build one for everybody." -- Coyote Osborne
"I'm afraid you have me mistaken for someone who can be shamed by a child." -- Quillspawn

Locked