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.

Sign up for GitHub

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

Jump to bottom

[l10n] Improve Brazilian Portuguese (pt-BR) locale #7850

Merged
merged 4 commits into from
Feb 8, 2023
Merged

[l10n] Improve Brazilian Portuguese (pt-BR) locale #7850

merged 4 commits into from
Feb 8, 2023

Conversation

ed-ateixeira
Copy link
Contributor

No description provided.

@ed-ateixeira
added toolbar titles translation for pt-BR
67ab7e9 
Signed-off-by: Eduardo Teixeira <69483392+ed-ateixeira@users.noreply.github.com>
@ed-ateixeira ed-ateixeira marked this pull request as draft February 7, 2023 13:10
@ed-ateixeira ed-ateixeira marked this pull request as ready for review February 7, 2023 13:10
@mui-bot
Copy link

mui-bot commented Feb 7, 2023 •
edited
Loading

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 491.8 843.8 655.8 653.62 137.113
Sort 100k rows ms 627.3 986.2 806.2 805.44 142.241
Select 100k rows ms 158.6 298.6 244.6 236.7 45.407
Deselect 100k rows ms 120.5 210.2 189.9 181.42 31.559

Generated by 🚫 dangerJS against 06868fe

@alexfauquette alexfauquette changed the title Added toolbar titles translation for pt-BR [l10n] Improve Portuguese (Brazil) (pt-BR) locale Feb 7, 2023
@alexfauquette alexfauquette added the l10n localization label Feb 7, 2023
@alexfauquette alexfauquette added the needs cherry-pick The PR should be cherry-picked to master after merge label Feb 7, 2023
m4theushw
m4theushw approved these changes Feb 7, 2023
View reviewed changes
Copy link
Member

@m4theushw m4theushw left a comment

Choose a reason for hiding this comment

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

Just a typo.

packages/x-date-pickers/src/locales/ptBR.ts Outdated Show resolved Hide resolved
@m4theushw m4theushw changed the title [l10n] Improve Portuguese (Brazil) (pt-BR) locale [l10n] Improve Brazilian Portuguese (pt-BR) locale Feb 7, 2023
m4theushw and others added 2 commits February 7, 2023 20:05
@m4theushw
Merge branch 'master' into HEAD
bf08ba9
@alexfauquette @m4theushw
Apply suggestions from code review
06868fe 
Co-authored-by: Matheus Wichman <matheushw@outlook.com>
Signed-off-by: Alexandre Fauquette <45398769+alexfauquette@users.noreply.github.com>
@alexfauquette alexfauquette merged commit 62cb082 into mui:master Feb 8, 2023
@alexfauquette
Copy link
Member

Thanks for the improvement :)

@alexfauquette alexfauquette removed the needs cherry-pick The PR should be cherry-picked to master after merge label Feb 8, 2023
@alexfauquette alexfauquette mentioned this pull request Feb 8, 2023
Merged
@dexamundsen dexamundsen mentioned this pull request Aug 31, 2024
Closed
@dexamundsen dexamundsen mentioned this pull request Sep 20, 2024
Closed
@dexamundsen dexamundsen mentioned this pull request Oct 4, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m4theushw m4theushw m4theushw approved these changes

@alexfauquette alexfauquette alexfauquette approved these changes

Assignees
No one assigned
Labels
l10n localization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ed-ateixeira @mui-bot @alexfauquette @m4theushw