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

Update on pt-br translation #21

Merged
merged 1 commit into from Dec 10, 2015
Merged

Conversation

breakzplatform
Copy link
Contributor

  • Fixed some semantic errors
  • Added more portuguese expressions and terms
  • Fixed some typos and ponctuation
  • Entire table of contents in portuguese (when needed)
  • Title of the page in portuguese

- Fixed some semantic errors
- Added more portuguese expressions and terms
- Fixed some typos and ponctuation
- Entire table of contents in portuguese (when needed)
- Title of the page in portuguese
@camsong
Copy link
Owner

camsong commented Dec 10, 2015

This anchor link is broken

image

@camsong
Copy link
Owner

camsong commented Dec 10, 2015

image
broken too

@breakzplatform
Copy link
Contributor Author

@camsong, something is wrong. Here on my repo (on the patch-1 branch) everything is working fine on my Chrome: https://github.com/joselitojunior/You-Dont-Need-jQuery/blob/patch-1/README.pt-BR.md

And, on my translation, there's no such thing as "CSS & Style". I'm using "CSS & Estilo", fully translated.
Are you seeing the appropriate file?

@camsong
Copy link
Owner

camsong commented Dec 10, 2015

@joselitojunior Yep, this PR is pretty good. My mistake to check the wrong file.
Great thanks for your contribution, please help to keep this up to date 👍

camsong added a commit that referenced this pull request Dec 10, 2015
@camsong camsong merged commit c96fe70 into camsong:master Dec 10, 2015
@breakzplatform breakzplatform deleted the patch-1 branch December 10, 2015 04:57
@breakzplatform
Copy link
Contributor Author

thx @camsong 🚀

camsong added a commit that referenced this pull request Jan 4, 2016
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.

None yet

2 participants