Skip to content

feat: add Simplified Chinese translation - #3619

Open
Tunglies wants to merge 8 commits into
gitify-app:mainfrom
Tunglies:i18n-chinese-simpified
Open

Tunglies wants to merge 8 commits into
gitify-app:mainfrom
Tunglies:i18n-chinese-simpified

Conversation

@Tunglies

@Tunglies Tunglies commented Nov 4, 2026 •

Copy link
Copy Markdown

Add Simplified Chinese (zh-Hans, 简体中文) for the renderer and application-owned native UI.

Provide the translation resources, language registration, Chinese duration and plural handling, and locale-specific tests. Match explicit script tags before region hints so Traditional Chinese system preferences do not select Simplified Chinese.

Preserve existing English wording, interpolation variables, component tags, and user-provided content.

Blocked by:

Merge after both prerequisites have landed. Native UI translation support from #3652 is included through #3653.

This Draft currently includes the prerequisite commits because all PRs target main. Review the Simplified Chinese change separately in this comparison. The branch will be rebased after the prerequisites merge.

Related: #1952


Add Simplified Chinese localization support

Related: #1952
Nice to have: #3463

Assisted by GPT6(high-effort)
Human confirmed(or adjust) the accuracy of the translation as a native speaker.
Some infrastructure might changes as needed

@github-actions github-actions Bot added the enhancement New feature or enhancement to existing functionality label Nov 4, 2026
@Tunglies
Tunglies force-pushed the i18n-chinese-simpified branch from 6392205 to cc73d61 Compare November 4, 2026 10:37
@setchy

This comment was marked as outdated.

@Tunglies
Tunglies force-pushed the i18n-chinese-simpified branch 4 times, most recently from f9e6090 to 61a4f55 Compare November 5, 2026 00:15
@Tunglies
Tunglies marked this pull request as ready for review November 5, 2026 00:17
@Tunglies

Tunglies commented Nov 5, 2026

Copy link
Copy Markdown
Author

There you go @setchy

@Tunglies
Tunglies force-pushed the i18n-chinese-simpified branch from 61a4f55 to 8976b80 Compare November 5, 2026 06:01

@setchy setchy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for working on the translations @Tunglies.

When ready, we may want to split this into a few separate smaller changes such as

- move i18n locales to shared
- support translate main
- support OS language sync
- simplified Chinese translation
- etc

@Tunglies

Tunglies commented Nov 6, 2026

Copy link
Copy Markdown
Author

More smaller commit? I see. :)

@setchy

setchy commented Nov 6, 2026

Copy link
Copy Markdown
Member

More smaller commit? I see. :)

independent PRs please. makes it easier to review and merge. If it was translation alone that is fine as a single PR, but all the other platform changes I think should be in their own independent PRs

@Tunglies
Tunglies marked this pull request as draft November 7, 2026 01:51
Attach window handlers before renderer loading and track menubar visibility so tray actions stay synchronized.
Share language metadata with tooling and the main process without importing renderer initialization.
Resolve application-generated descriptions at display time and pass the active locale to date rendering while preserving existing English wording.
Route application-owned native text through a separate i18next instance and include it in extraction without changing English output.
Resolve and persist language in the main process before native UI and renderer startup, so both processes use the same authoritative preference.
Match explicit script tags before region hints so Traditional Chinese system preferences do not select Simplified Chinese.

Ref gitify-app#1952
@Tunglies
Tunglies force-pushed the i18n-chinese-simpified branch from 8976b80 to cf3749c Compare November 7, 2026 02:22
@Tunglies Tunglies changed the title feat: add Simplified Chinese localization feat: add Simplified Chinese translation Nov 7, 2026
@Tunglies
Tunglies marked this pull request as ready for review November 7, 2026 02:25

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement to existing functionality

Development

Successfully merging this pull request may close these issues.

2 participants