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

Including Missing translation for pt-br (pt-br string.js) #7470

Merged
merged 2 commits into from
Apr 14, 2014

Conversation

Rynaro
Copy link
Contributor

@Rynaro Rynaro commented Apr 10, 2014

Added line for "CMD_OPEN_PREFERENCES" in Brazilian Portuguese language.

Added line for "CMD_OPEN_PREFERENCES" in Brazilian Portuguese language.
@njx njx self-assigned this Apr 11, 2014
@njx
Copy link
Contributor

njx commented Apr 11, 2014

@MarkLinus @rodrigost23 Would you mind reviewing this small change?

@Rynaro - I know this is a small pull request, but our process requires everyone to sign a CLA in order to have a contribution accepted. Could you sign it at http://dev.brackets.io/brackets-contributor-license-agreement.html? Thanks!

@Rynaro
Copy link
Contributor Author

Rynaro commented Apr 11, 2014

Thank you very much.
I've done what you asked me.
Should I resubmit my commit?

@njx
Copy link
Contributor

njx commented Apr 11, 2014

@Rynaro No, you don't need to resubmit - thanks for signing. Once someone else has reviewed this string I can merge it. (I might resubmit it against the release branch if we get a review soon enough so it gets into the current release.) Thanks!

@@ -296,6 +296,7 @@ define({
"CMD_SHOW_EXTENSIONS_FOLDER" : "Mostrar pasta de extensões",
"CMD_TWITTER" : "{TWITTER_NAME} no Twitter",
"CMD_ABOUT" : "Sobre o {APP_TITLE}",
"CMD_OPEN_PREFERENCES" : "Abrir preferências do arquivo",
Copy link
Contributor

Choose a reason for hiding this comment

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

Well, I think "Abrir arquivo de configurações" is a more accurate translation considering the context.

@Rynaro
Copy link
Contributor Author

Rynaro commented Apr 12, 2014

Adjustments so that the user can better understand what is written on the item.

@njx
Copy link
Contributor

njx commented Apr 14, 2014

Looks good, thanks @Rynaro and @rodrigost23

njx added a commit that referenced this pull request Apr 14, 2014
Including Missing translation for pt-br (pt-br string.js)
@njx njx merged commit 79df1a0 into adobe:master Apr 14, 2014
@njx
Copy link
Contributor

njx commented Apr 14, 2014

FYI - this didn't make it into release 38 but will be in the next one (release 39).

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.

3 participants