Skip to content

[l10n] Add missing MuiPagination localization to zh-CN locale#48741

Open
greymoth-jp wants to merge 1 commit into
mui:masterfrom
greymoth-jp:l10n/zhcn-mui-pagination
Open

[l10n] Add missing MuiPagination localization to zh-CN locale#48741
greymoth-jp wants to merge 1 commit into
mui:masterfrom
greymoth-jp:l10n/zhcn-mui-pagination

Conversation

@greymoth-jp

Copy link
Copy Markdown

The zh-CN locale is missing the MuiPagination block, so its pagination aria-label and the getItemAriaLabel text fall back to English even though every other component in the file is localized.

MuiPagination (aria-label plus getItemAriaLabel) is part of the canonical localizable set documented in enUS.ts, and almost every locale file already provides it. The only ones that don't are the three Chinese locales (zh-CN, zh-HK, zh-TW); for comparison ja-JP and ko-KR both localize this block already. So this looks like a copy-paste gap rather than an intentional omission.

This PR fills it in for zh-CN. For the first/last/next/previous and page wording I reused the noun phrases already present in this file's MuiTablePagination.getItemAriaLabel (第一页 / 最后一页 / 下一页 / 上一页) and added 转到 ("Go to") so it matches the English source, which marks every navigable item with "Go to" and leaves the current page unmarked.

Note: the same block is also absent from zh-HK and zh-TW. I left those out here since the Traditional Chinese wording differs and I would rather not guess at it. Happy to follow up if you want them covered too.

@code-infra-dashboard

Copy link
Copy Markdown

Deploy preview

https://deploy-preview-48741--material-ui.netlify.app/
QR code for https://deploy-preview-48741--material-ui.netlify.app/

Bundle size

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/private-theming 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

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.

1 participant