Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Added Brazilian Portuguese translation #1660

Merged
merged 3 commits into from
Oct 4, 2012
Merged

Conversation

max-giro
Copy link
Contributor

Created src/nls/pt-br/strings.js and src/nls/pt-br/urls.js

@ghost ghost assigned jasonsanjose Sep 17, 2012
@jasonsanjose
Copy link
Member

Assigning to me @jasonsanjose. We might not be able to get this in before our end of sprint today 9/17. @massimiliano-giroldi please sign the contributor license agreement here http://dev.brackets.io/brackets-contributor-license-agreement.html and reply when it's submitted. Thanks!

@max-giro
Copy link
Contributor Author

Ops, done!


define({
// Relative to the samples folder
"GETTING_STARTED" : "pt-br/Primeiros Passos"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You changed the path here but the files are under samples/pt-br/Getting Started/.... Please rename the Getting Started folder to match.

@jasonsanjose
Copy link
Member

Looks good, just some minor comments.


"LIVE_DEVELOPMENT_ERROR_TITLE" : "Erro de Live Preview",
"LIVE_DEVELOPMENT_RELAUNCH_TITLE" : "Conectando ao navegador",
"LIVE_DEVELOPMENT_ERROR_MESSAGE" : "Para que a Live Preview conecte, Chrome deve ser aberto com a opção de depuração remota ativada.<br /><br />Gostaria de reabir Chrome e permitir a depuração remota?",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo reabir to reabrir?

@max-giro
Copy link
Contributor Author

max-giro commented Oct 4, 2012

Thank you for pointing out the typos and the path error.
I pushed a new fixed version.

@jasonsanjose
Copy link
Member

Looks good. Merging.

jasonsanjose added a commit that referenced this pull request Oct 4, 2012
Added Brazilian Portuguese translation
@jasonsanjose jasonsanjose merged commit b2ca146 into adobe:master Oct 4, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants