CrowdIn is set up

The l10n of Pale Moon. Rawr.

Moderator: satrow

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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 postby dark_moon » Wed, 09 Nov 2016, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 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

dark_moon

Re: CrowdIn is set up

Unread postby dark_moon » Wed, 09 Nov 2016, 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: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 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

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: Thu, 27 Aug 2015, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread postby GMforker » Wed, 09 Nov 2016, 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: 613
Joined: Tue, 01 Jan 2013, 10:46
Location: Sudamérica
Contact:

Re: CrowdIn is set up

Unread postby KNTRO » Wed, 09 Nov 2016, 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
Image
Our pale faces sink in time
Either way it's suicide

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 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

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 21:35

Here's the result of CrowdIn's build+save process for Russian
Attachments
ru.zip
(446.09 KiB) Downloaded 29 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

User avatar
JustOff
Localization Coordinator
Localization Coordinator
Posts: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: Thu, 27 Aug 2015, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread postby GMforker » Wed, 09 Nov 2016, 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: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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 & -> &amp;, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 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

GMforker
Astronaut
Astronaut
Posts: 537
Joined: Thu, 27 Aug 2015, 06:29
Location: Czech Republic

Re: CrowdIn is set up

Unread postby GMforker » Wed, 09 Nov 2016, 22:39

JustOff wrote:This looks like & -> &amp;, 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: 1423
Joined: Thu, 03 Sep 2015, 19:47
Location: UA
Contact:

Re: CrowdIn is set up

Unread postby JustOff » Wed, 09 Nov 2016, 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: 21705
Joined: Sun, 28 Aug 2011, 17:27
Location: 58.5°N 15.5°E
Contact:

Re: CrowdIn is set up

Unread postby Moonchild » Wed, 09 Nov 2016, 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


Return to “Localization”

Who is online

Users browsing this forum: No registered users and 4 guests