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

Polish translation for web client #2232

Merged
merged 2 commits into from
Mar 14, 2016

Conversation

pstekieli
Copy link

This is a polish translation for web client module of OTP. There might
some issues with spacing that do not strictly depend on the language file itself.
Even though, everything remains readable, and do not affect the general
construction of the client.

This is a polish translation for web client module of OTP. There are
some issues with spacing that do not depend on the language file itself.
Even though, everything remains readable, and do not affect the general
construction of the client.
@pstekieli pstekieli changed the title [Private] Polish translation for web client Polish translation for web client Mar 14, 2016
@abyrd
Copy link
Member

abyrd commented Mar 14, 2016

Hi @Ergone, looks good except that there are a lot of whitespace changes in config.js. Can you remove those from this PR?

I reused config.js from master branch just to modify this single line and add Polish language support. The rest of lines remains the same as they are in the original file.
@pstekieli
Copy link
Author

Hi! I've just edited config.js file. Do you want me to modify index.html as well ?

@abyrd
Copy link
Member

abyrd commented Mar 14, 2016

No need, it's much better now. In index.html it's still clear where the important changes are.

abyrd added a commit that referenced this pull request Mar 14, 2016
@abyrd abyrd merged commit 9499ba2 into opentripplanner:master Mar 14, 2016
@pstekieli
Copy link
Author

Thanks 👍

@pstekieli pstekieli deleted the polish-translation branch March 14, 2016 16:57
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.

2 participants