Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update Portuguese translator #236

Merged
merged 8 commits into from
Nov 30, 2024
Merged

Update Portuguese translator #236

merged 8 commits into from
Nov 30, 2024

Conversation

MestreWalla
Copy link
Contributor

Better context, add some missing translation

- Fix typo in location coordinates (Aletitude -> Latitude)
- Standardize UI text patterns using 'Mostrar' instead of 'Ativar/desativar'
- Make translations more natural and consistent with English version
- Update digital clock and shortcuts related translations
- Remove machine translation comment
- Enhances clarity and naturalness of PT-BR translations
- Fixes spelling inconsistencies (e.g., "Ícone")
- Standardizes technical terms for better understanding
- Adds missing translations for AI tools
- Improves text formatting in Japanese and Hindi
Copy link
Collaborator

@itz-rj-here itz-rj-here left a comment

Choose a reason for hiding this comment

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

Ohh It's ok np.

@@ -1594,7 +1620,7 @@ function applyLanguage(lang) {
{ id: 'perplexity', key: 'perplexity' },
{ id: 'firefly', key: 'firefly' },
{ id: 'github', key: 'github' },
{ id: 'googleAppsHover', key: 'googleAppsHover' },
{ id: 'googleAppsHover', key: 'googleAppsHover' }
Copy link
Collaborator

Choose a reason for hiding this comment

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

?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why you removed a comma from the code?

Copy link
Owner

Choose a reason for hiding this comment

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

If this is last element commas are not needed

Copy link
Collaborator

Choose a reason for hiding this comment

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

It's fine, it's not really needed, doesn't affect anything

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah I was reviewing the code from my phone, thats way i didn't saw the last line sorry 😅

@XengShi XengShi merged commit abfbde4 into XengShi:main Nov 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants