Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding localization #6

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Adding localization #6

wants to merge 29 commits into from

Conversation

remil19
Copy link

@remil19 remil19 commented Mar 2, 2014

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.

@heyimjuani
Copy link
Contributor

This is f*cking awesome

On 02/03/2014, at 04:04, Rémi Lapeyre [email protected] 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:

#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
Copy link
Author

remil19 commented Mar 3, 2014

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

@remil19 remil19 closed this Mar 3, 2014
@remil19 remil19 reopened this Mar 3, 2014
@fmpaes
Copy link

fmpaes commented Mar 5, 2014

+1 for support

@argen
Copy link
Contributor

argen commented Mar 7, 2014

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

@argen
Copy link
Contributor

argen commented Mar 7, 2014

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

@EvandroZanatta
Copy link

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
Copy link
Author

remil19 commented Mar 10, 2014

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
Copy link

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

@c0b41
Copy link

c0b41 commented Mar 10, 2014

@remil19 faq page generate translate?

@remil19
Copy link
Author

remil19 commented Mar 10, 2014

@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
Copy link

I found some errors still exist, I will review.

I have reviewed the translation of every sentence trying to give more sense to the sentences and making the most natual reading. 
Some grammatical errors have also been corrected.
@bouiboui bouiboui mentioned this pull request Mar 11, 2014
@c0b41
Copy link

c0b41 commented Mar 11, 2014

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

@remil19
Copy link
Author

remil19 commented Mar 11, 2014

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 and others added 7 commits March 11, 2014 15:30
Show the "readme" file the steps that must be followed to correctly translate the site.
Merge branch 'master' of github.com:remil19/popcorn-time.github.io

Conflicts:
	README.md
@remil19
Copy link
Author

remil19 commented Mar 12, 2014

@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
Copy link

@remil19 +1

1 similar comment
@fmpaes
Copy link

fmpaes commented Mar 13, 2014

@remil19 +1

Felipe Mário and others added 4 commits March 13, 2014 03:48
@EvandroZanatta
Copy link

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

@remil19
Copy link
Author

remil19 commented Mar 13, 2014

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
Copy link
Author

remil19 commented Mar 13, 2014

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
Copy link

it should have been fused to the main repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants