Closed msales78 closed 7 years ago
Hello @msales78, apologies for the delay. We need to review your pull request first. Will let you know when that's done. Thanks.
Thanks @karypun, no problem, i am available for any questions. :)
@msales78 Sorry for the delay in this. We had it reviewed and it's great. There were a few points made though, first one is quite important the second one is up to your discretion:
2a. on deaf_pt-BR: Not wrong either but it says : Projetando para usuários que são deficientes auditivos ou surdos. I think the "que são" is superfluous. See this link from a Microsoft portuguese page for example: https://msdn.microsoft.com/pt-br/bb899566 it says only "Accessibilidade para usuários deficientes visuais" , the "que são" is not present. The difference in english would be "projecting for hearing impared users" Vs "projecting for users that are hearing impared" .
2b. low-vision_pt-BR: same as above in the title.
Other than that is all mint :)
@karypun Thank you. No problems with the delay. The points make sense and I'll make the adjustments later this week. I'll send it next. Thanks.
Hello, I translated the posters into Brazilian Portuguese. It's my first pull request, so I do not know if I did something wrong or if I did everything right. Sorry if I missed something ... thanks for allowing everyone's contribution.