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.

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

[R] Tips 翻译提交 #1916

Merged
merged 58 commits into from
May 7, 2022
Merged

[R] Tips 翻译提交 #1916

merged 58 commits into from
May 7, 2022

Conversation

AnzhiZhang
Copy link
Contributor

  • 我已仔细阅读注意事项 CONTRIBUTING
  • 我已对 PR 作出 标记
  • 我已确认英文原文(如 en_us.json)存在且完整,内容与中文对应;
  • 我已确认提交文件的路径名称正确例子);
    • 如果是 1.12 翻译,应该是:projects/1.12/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.lang
    • 如果是 1.16 及以上的翻译,应该是:projects/{版本}/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.json
  • 我已阅读并同意许可协议:知识共享 署名-非商业性使用-相同方式共享 4.0 国际许可协议
  • 刷新 PR 的标签/状态,有需要再点击;

@boring-cyborg boring-cyborg bot added 1.16.x 1.18.x Forge include Forge mod labels May 1, 2022
@cfpa-bot
Copy link
Contributor

cfpa-bot bot commented May 1, 2022

@CFPABot CFPABot added 100+ Size label. need review labels May 1, 2022
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.18/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.18/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
AnzhiZhang and others added 4 commits May 2, 2022 01:24
Co-authored-by: n0099 <n@n0099.net>
Co-authored-by: n0099 <n@n0099.net>
Co-authored-by: n0099 <n@n0099.net>
Co-authored-by: n0099 <n@n0099.net>
@n0099 n0099 changed the title [R]Tips 翻译提交 [R] Tips 翻译提交 May 1, 2022
Copy link
Contributor

@Rinko1231 Rinko1231 left a comment

Choose a reason for hiding this comment

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

语序有点怪

projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
AnzhiZhang and others added 9 commits May 2, 2022 15:15
Co-authored-by: Rinko1231 <89371859+Rinko1231@users.noreply.github.com>
Co-authored-by: Rinko1231 <89371859+Rinko1231@users.noreply.github.com>
Co-authored-by: Rinko1231 <89371859+Rinko1231@users.noreply.github.com>
Co-authored-by: Rinko1231 <89371859+Rinko1231@users.noreply.github.com>
Co-authored-by: Rinko1231 <89371859+Rinko1231@users.noreply.github.com>
AnzhiZhang and others added 3 commits May 6, 2022 07:37
Co-authored-by: n0099 <n@n0099.net>
Co-authored-by: n0099 <n@n0099.net>
Copy link
Contributor Author

@AnzhiZhang AnzhiZhang left a comment

Choose a reason for hiding this comment

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

改了一些细节和错别字,可以的话直接commit就行,我更新1.18

projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
"tips.tip.smoke_bees": "在蜂巢下方放置营火会使蜜蜂不被激怒。",
"tips.tip.lava_fuel": "熔岩桶可以作为熔炉的燃料使用。",
"tips.tip.wooden_tool_fuel": "不再使用的木制工具能够作为熔炉燃料使用。",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
"tips.tip.wooden_tool_fuel": "不再使用的木制工具能够作为熔炉燃料使用",
"tips.tip.wooden_tool_fuel": "不再使用的木质工具能够作为熔炉燃料使用",

Copy link
Member

Choose a reason for hiding this comment

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

木制工具
虽然原版用的木质工具

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个质是材质的意思
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

如果木制,指的意思“木头制造的工具”,还是木头材质的
查了下词典,木制应该是不能作形容词使用的
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

与这个相关的还有19行的那条

Copy link
Member

Choose a reason for hiding this comment

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

关于用X制还是X质,CFPA 近期的一次讨论的共识是使用前者或不用 zhi,规避它,比如“钻石工具”。

在这次讨论前,CFPA前管理的共识基本也是这样,可以从 commit 记录中看出。

另见清秋观点:#1528 (comment)

有待进一步讨论。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这次提交的zhi工具,不适用不用zhi的方式,查询了一下质的意思和用法
image

Copy link
Contributor Author

@AnzhiZhang AnzhiZhang May 6, 2022

Choose a reason for hiding this comment

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

我看了一下两个字的历史和说文解字解释
说文解字

卷四,刀部
image
裁也。从 刀、从 未。未,物成,有滋味,可裁斷。一曰止也。𠝁,古文制如此。

卷六,贝部
image
以物相贅。从貝,从斦。闕。


由此得知,质没有制造的意思,材质的意思是现代引申的,而制本意的制造是合适的。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

目前已修改为,以上仅为原理解释

Copy link
Member

@mamaruo mamaruo May 6, 2022

Choose a reason for hiding this comment

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

一个简易论证,根据搜索结果易得此处应用“制”:
铁质_百度搜索铁质 - Google 搜索
铁制_百度搜索铁制 - Google 搜索

恕我实在不是很能看懂《说文解字》,而且我今天想摆烂。

2022年5月9日编辑:unresolve 了这个 review,方便之后找

projects/1.16/assets/tips/tips/lang/zh_cn.json Outdated Show resolved Hide resolved
Co-authored-by: Andy Zhang <37402126+AnzhiZhang@users.noreply.github.com>
@AnzhiZhang
Copy link
Contributor Author

发现这个mod根本没有1.18,仓库里的1.18语言文件是1.16复制的吧

@n0099
Copy link
Contributor

n0099 commented May 6, 2022

@AnzhiZhang AnzhiZhang requested a review from mamaruo May 6, 2022 01:02
@AnzhiZhang AnzhiZhang requested a review from Cyl18 May 6, 2022 15:47
@mamaruo mamaruo added the 即将合并 在此标签被添加的 24 小时后 Pull Request 才可被合并 label May 6, 2022
@n0099 n0099 merged commit 3211c98 into CFPAOrg:main May 7, 2022
@AnzhiZhang AnzhiZhang deleted the tips branch May 7, 2022 07:34
@mamaruo mamaruo mentioned this pull request May 7, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.16.x 1.18.x 100+ Size label. Forge include Forge mod 即将合并 在此标签被添加的 24 小时后 Pull Request 才可被合并
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants