Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update values-pt (Portuguese) #199

Merged
merged 1 commit into from
Jan 2, 2025
Merged

Conversation

TheLilUser
Copy link
Contributor

@TheLilUser TheLilUser commented Dec 1, 2024

Hi, I am new in this repo and I am Portuguese-native (from Brazil). Here I bring some of the small corrections on the translations and their respective details why they have been changed.


⚙️ Changes

  1. cantread: "Impossível" is correct, but not commonly used or recognised in prompt message translations from "cannot" or "unable", then "Não é possível" fits a lot better here. "Configurações" is commonly used because there's usually more than a setting.
  2. script_1E00: To match with other strings, I've changed from "Latim Expandido Adicional" to "Expansão Adicional do Latim" that looks more polished.
  3. script_3000: For more than a punctuation, we use "pontuações", as that Unicode block contains multiple CJK punctuations.
  4. script_FE20: Translation doesn't seem to be right and doesn't fit with script_0300. "Marcas Combinantes do Meio" is a clean choice here.

ℹ️ Additional informations

This is my very first contribution attempt here, then I decided to make just a few important changes into the values-pt file. I may submit other pull requests about diffs into the Portuguese file if this gets approved. If there's any additional change to be made, please let me know, any change request is welcomed.

I use UnicodePad at my daily routine to pick up characters that do not appear in my keyboard, then decided to check out the code for it.


@TheLilUser TheLilUser changed the title values-pt Update values-pt (Portuguese) Dec 1, 2024
@Ryosuke839
Copy link
Owner

Thank you for contributing. Everything look good to me! (as a non Portuguese speaker)

@Ryosuke839 Ryosuke839 merged commit 9bb5a3a into Ryosuke839:develop Jan 2, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants