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

Update index.html #5345

Merged
merged 2 commits into from
Nov 28, 2013
Merged

Update index.html #5345

merged 2 commits into from
Nov 28, 2013

Conversation

RafaelOlivra
Copy link
Contributor

Translated the alternative image text to pt_BR.
From "A screenshot showing CSS Quick Edit" to the equivalent "Uma screenshot mostrando o editor rápido de CSS"

Translated the alternative image text to pt_BR.
From "A screenshot showing CSS Quick Edit" to the equivalent "Uma screenshot mostrando o editor rápido de CSS"
@ghost ghost assigned TomMalbran Sep 26, 2013
@kkarlesk
Copy link
Contributor

@TomMalbran, are you able to address this one?

@TomMalbran
Copy link
Contributor

@kkarlesk No, I don't know Portuguese.

@RafaelOlivra
Copy link
Contributor Author

Is there anyone who speak Portuguese working on brackets?

@TomMalbran
Copy link
Contributor

I guess no one reassigned this PR. Well, I think the string is good enough, but just in case: @MarkLinus, @diegoleme, @rodrigost23 or @vinigordex. Can any of you review this? Thanks

@@ -47,7 +47,7 @@
</samp>

<a href="screenshots/quick-edit.png">
<img alt="A screenshot showing CSS Quick Edit" src="screenshots/quick-edit.png" />
<img alt="Uma screenshot mostrando o editor rápido de CSS" src="screenshots/quick-edit.png" />

Choose a reason for hiding this comment

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

I think "Um screenshot ..." is more adequate

@danilo-valente
Copy link

I've just reviewed the code. Disregarding the note above, all the rest is ok

@TomMalbran
Copy link
Contributor

@MarkLinus Thanks. @RafaelOlivra Could you make that change?

Done!
Changed "Uma" to "Um".
@danilo-valente
Copy link

Now this pull request can be accepted

@TomMalbran
Copy link
Contributor

Great, thanks. One final thing before I can merge this. Can you sign up the contributor agreement http://dev.brackets.io/brackets-contributor-license-agreement.html? Thanks

@RafaelOlivra
Copy link
Contributor Author

@TomMalbran Done :)

@TomMalbran
Copy link
Contributor

Thanks. Merging now. Sorry for the delay, I was expecting it to be reassigned to someone else.

TomMalbran added a commit that referenced this pull request Nov 28, 2013
@TomMalbran TomMalbran merged commit 81cfc61 into adobe:master Nov 28, 2013
@RafaelOlivra RafaelOlivra deleted the patch-1 branch November 28, 2013 23:46
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.

4 participants