CrowdIn is set up

The l10n of Pale Moon. Rawr.
JustOff

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.

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: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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?
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

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%

JustOff

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?

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

JustOff

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.

GMforker

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.

KNTRO

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

JustOff

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)

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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?
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

JustOff

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.

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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 42 times
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

JustOff

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.

GMforker

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...

JustOff

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.

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

GMforker

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.

JustOff

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)

User avatar
Moonchild
Pale Moon guru
Pale Moon guru
Posts: 35474
Joined: 2011-08-28, 17:27
Location: Motala, SE
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.
"Sometimes, the best way to get what you want is to be a good person." -- Louis Rossmann
"Seek wisdom, not knowledge. Knowledge is of the past; wisdom is of the future." -- Native American proverb
"Linux makes everything difficult." -- Lyceus Anubite

Locked