Skip to content

feat(i18n): translate GUI search and library actions - #953

Merged
LargeModGames merged 1 commit into
mainfrom
feat/l10n-zh-gui-library
Oct 28, 2026
Merged

LargeModGames merged 1 commit into
mainfrom
feat/l10n-zh-gui-library

Conversation

@2570165831

@2570165831 2570165831 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

The eleventh Chinese translation PR (refs #726), the second of the GUI table: the browser frontend's search and command bar, Discover, the library menus and sorting, playlist sync, radio actions, the update notices and the keyboard hints, in Simplified Chinese (zh-Hans) and Traditional Chinese with Taiwan wording (zh-Hant).

  • src/core/i18n/gui/zh_hans.rs, zh_hant.rs: 170 more strings, so 384 of the GUI table's 510 now have both translations.
  • The remaining 126 (statistics and social views) stay on ..EN and come in the next PR.
  • Templates keep their placeholders, so names, queries, counts, versions and error text are shown exactly as given. Keys in the hint strings (enter, tab, shift+enter, J K, [ ]) and command tokens stay as typed. Quotation marks follow each script's existing entries: “…” in Simplified, 「…」 in Traditional.
  • gui/src/bindings/strings.json is regenerated.

English is unchanged.

Testing

macOS (aarch64), on top of fb8fbbb0:

  • cargo fmt --all --check: clean. cargo clippy --locked -- -D warnings: clean on slim (telemetry,tui) and default.
  • cargo test --locked: slim 1789 passed; default plus local-files,subsonic 2393 passed, 21 ignored; the gui feature set with the five sources 2202 passed, 30 ignored.
  • gui/: npm test 540 passed (61 files); npm run typecheck and npm run lint clean.

Not run: the Linux-only all-sources leg (PipeWire), which CI covers.

Additional notes

A native-speaker review of the wording is welcome; any fix is a one-line change in the table.

Summary by CodeRabbit

  • Localization
    • Expanded Simplified and Traditional Chinese translations across the interface, including discovery and search, playlists and library actions, sorting, listening history, song radio, and version updates.
    • Added localized labels and prompts for explicit content, Spotify attribution, personalized selections, and deleting playlists or library items.

Translate browser search, discovery, playlist sync, library menus, sorting and radio actions into both Chinese scripts. Keep action payloads and command/config tokens unchanged.
@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between fb8fbbb and 80807d8.

Files ignored due to path filters: 1
  • gui/src/bindings/strings.json is excluded by !gui/src/bindings/**
Files selected for processing: 2
  • src/core/i18n/gui/zh_hans.rs
  • src/core/i18n/gui/zh_hant.rs

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Walkthrough

The Simplified and Traditional Chinese GUI translation tables add labels for discovery, search, commands, library actions, listening history, deletion prompts, and sorting. Both tables retain English fallback declarations.

Changes: Chinese GUI translations

Layer / File(s) Summary
Discovery, search, and commands
src/core/i18n/gui/zh_hans.rs, src/core/i18n/gui/zh_hant.rs
Both locale tables add general labels and translations for discovery, search, command prompts, and command descriptions.
Search results and library actions
src/core/i18n/gui/zh_hans.rs, src/core/i18n/gui/zh_hant.rs
Both locale tables add version and search-result labels, track options, library actions, and album and artist sorting labels.
History, deletion, and sorting
src/core/i18n/gui/zh_hans.rs, src/core/i18n/gui/zh_hant.rs
Both locale tables add song radio, listening history, deletion prompts, and sorting options.

Priority: Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Possibly related issues

  • Issue 872: This work adds Simplified and Traditional Chinese GUI translations, matching the issue’s contributor-credit context for translation.

Possibly related PRs

Merge Risk: Minimal · up to 80807

No actionable merge-blocking risk is evident in the translated GUI strings.

Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title uses the allowed conventional-commit prefix feat(i18n): and clearly describes the main translation changes for GUI search and library actions. The subject is concise and imperative.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
✨ Simplify code
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@LargeModGames
LargeModGames merged commit ab58ce7 into main Oct 28, 2026
39 checks passed
@LargeModGames
LargeModGames deleted the feat/l10n-zh-gui-library branch October 28, 2026 20:32
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.

2 participants