popcorn-time / popcorn-time.github.io

Site for getpopcornti.me
getpopcornti.me
246 stars 199 forks source link

Adding localization #6

Open remil19 opened 10 years ago

remil19 commented 10 years ago

It's now very simple to translate the web site:

Any modification at the site should be done at dev/index.php

A small server-side program should determine which language choose for the visitor and redirect to the correct file.

heyimjuani commented 10 years ago

This is f*cking awesome

On 02/03/2014, at 04:04, Rémi Lapeyre notifications@github.com wrote:

It's now very simple to translate the web site:

first create a new file languages/'your lang'.json based on en.json; add your language at the first lines of dev/generate-static.php; run php generate-static.php in dev/ a new file for your language should be present at the root. Any modification at the site should be done at dev/index.php

A small server-side program should determine which language choose for the visitor and redirect to the correct file.

You can merge this Pull Request by running

git pull https://github.com/remil19/popcorn-time.github.io master Or view, comment on, or merge it at:

https://github.com/popcorn-time/popcorn-time.github.io/pull/6

Commit Summary

Create index.php Create generate-static.php Create en.json Create fr.json French translation generated File Changes

A dev/generate-static.php (29) A dev/index.php (204) A fr.html (201) M index.html (16) A languages/en.json (27) A languages/fr.json (27) Patch Links:

https://github.com/popcorn-time/popcorn-time.github.io/pull/6.patch https://github.com/popcorn-time/popcorn-time.github.io/pull/6.diff — Reply to this email directly or view it on GitHub.

remil19 commented 10 years ago

I've just seen that you're using Github Pages and i think it's only static so maybe you should add flags in order to let the user choose his language or use javascript to redirect to the correct page (remember, any change must be done in dev/index.php, any change in index.html will be over will be overwrited).

fmpaes commented 10 years ago

+1 for support

argen commented 10 years ago

Will help translate to spanish and make some amends for the english version.

argen commented 10 years ago

@abadfederico can we resolve conflicts and merge this so we can start translating?

EvandroZanatta commented 10 years ago

had cited an example( #17 ) with translated versions of the site, but I believe that using JSON files with the translations would be the best way. Just as it is translated the "App".

remil19 commented 10 years ago

The Turkish a-has just been added. Is there any problem with the content of this pull request because it's been some time and translations could be a plus to the site.

EvandroZanatta commented 10 years ago

@remil19 I will create a "pt-br.json" and translate ok?

c0b41 commented 10 years ago

@remil19 faq page generate translate?

remil19 commented 10 years ago

@cobaimelan there is just few lines to add in order to translate the faq, i will write them soon if i have time to.

@EvandroZanatta Thanks for that i just merged the changes .

EvandroZanatta commented 10 years ago

I found some errors still exist, I will review.

c0b41 commented 10 years ago

@remil19 No wonder it possible to change the font Turkish characters adsz

remil19 commented 10 years ago

I've seen that too. I'm not sure that it's the translating engine that should take care of that. Should we change the font for all languages in order to keep a unity or just change the font for the Turkish even if it could be a bit more complicated to maintain?

Moreover when i first forked the project in order to add localization, there was no faq, i can easily add it now but maybe the changes already made should be merged before, that would simpler.

I've noticed that there is an Spanish translation in the main repo, is this pull request still usefull or should i delete it?

remil19 commented 10 years ago

@heyimjuani is has been 11 days since this pull request has been opened. There is more differences between the two repositories with time passing. Is the merge between them consider or should we stop to work on the fork?

EvandroZanatta commented 10 years ago

@remil19 +1

fmpaes commented 10 years ago

@remil19 +1

EvandroZanatta commented 10 years ago

would not be necessary to update the download links? on the official website, the link to download the linux version is broken

remil19 commented 10 years ago

There has been a lot of modifications in index.html the last twelve days but if we update the page on my repository it will still have to be updated every time a modification is made is the official repository so i think we should only update index.php when the merge will be done.

remil19 commented 10 years ago

There is also some issues specific to the translations such as the one @cobaimelan raised (it happens also for Greek), another is some sentences are longer once translated so their end is hide by an image put next to it. I cannot make modifications impacting the design of the site, the official team have to make them (which font use, should we move the image or break the lines which are too long...)

It' s actually up to the official team now.

EvandroZanatta commented 10 years ago

it should have been fused to the main repository.