Skip to content

优化陶瓦联机反馈提示#6120

Open
CiiLu wants to merge 2 commits into
HMCL-dev:mainfrom
CiiLu:ee
Open

优化陶瓦联机反馈提示#6120
CiiLu wants to merge 2 commits into
HMCL-dev:mainfrom
CiiLu:ee

Conversation

@CiiLu
Copy link
Copy Markdown
Contributor

@CiiLu CiiLu commented May 17, 2026

右侧列表内的是随机展示的,和更新没关系
image

@Glavo
Copy link
Copy Markdown
Member

Glavo commented May 18, 2026

/gemini review

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the feedback confirmation dialog to use a new localization key, terracotta.feedback.desc.update, and introduces a shorter version for the existing terracotta.feedback.desc key across several language files. A critical issue was identified in the Arabic localization file where the original key was renamed but not replaced, potentially leading to a MissingResourceException if the old key is still required by the application.

terracotta.back=خروج
terracotta.feedback.title=ملء استمارة التقييم
terracotta.feedback.desc=مع تحديث HMCL لنواة متعدد اللاعبين، نأمل أن تخصص 10 ثوانٍ لملء استمارة التقييم.
terracotta.feedback.desc.update=مع تحديث HMCL لنواة متعدد اللاعبين، نأمل أن تخصص 10 ثوانٍ لملء استمارة التقييم.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

high

此处的修改导致 terracotta.feedback.desc 键在阿拉伯语本地化文件中缺失。这会在运行时导致 MissingResourceException,因为代码中仍然有地方在使用这个键。

您应该像其他本地化文件一样,在重命名此键的同时,添加一个新的 terracotta.feedback.desc 键,其值为通用的反馈提示。

terracotta.feedback.desc=نأمل أن تخصص 10 ثوانٍ لملء استمارة التقييم.
terracotta.feedback.desc.update=مع تحديث HMCL لنواة متعدد اللاعبين، نأمل أن تخصص 10 ثوانٍ لملء استمارة التقييم.

@Glavo Glavo requested a review from burningtnt May 18, 2026 12:32
Copy link
Copy Markdown
Member

@burningtnt burningtnt left a comment

Choose a reason for hiding this comment

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

没有看出这样改的必要,但同样没有想到不这样改的必要

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.

3 participants