-
Notifications
You must be signed in to change notification settings - Fork 15
New Crowdin updates #90
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
Conversation
审阅者指南此 PR 集成了 Crowdin 本地化更新,优化了西班牙语和俄语翻译,更正了链接路径,更新了组件区域设置 JSON,并对文档进行了细微的格式调整。 文件级别更改
提示和命令与 Sourcery 交互
自定义您的体验访问您的 仪表盘 以:
获取帮助Original review guide in EnglishReviewer's GuideThis PR integrates Crowdin localization updates, refining Spanish and Russian translations, correcting link paths, updating component locale JSON, and making minor formatting adjustments across the docs. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Gzh0821 - 我已经审阅了你的更改 - 以下是一些反馈:
- 不要翻译CSS属性或值(例如,‘filter: brillo(60%)’ 是无效的CSS——请保留原始的 ‘brightness(60%)’)。
- 俄罗斯语的
submitlevel.md
现在包含英文文本而不是俄语本地化内容——请恢复正确的翻译内容。 - 仔细检查本地化指南中更新的相对链接,确保它们仍然解析到正确的页面。
给AI代理的提示
请处理此代码审查中的评论:
## 整体评论
- 不要翻译CSS属性或值(例如,‘filter: brillo(60%)’ 是无效的CSS——请保留原始的 ‘brightness(60%)’)。
- 俄罗斯语的 `submitlevel.md` 现在包含英文文本而不是俄语本地化内容——请恢复正确的翻译内容。
- 仔细检查本地化指南中更新的相对链接,确保它们仍然解析到正确的页面。
## 独立评论
### 评论 1
<location> `src/es-ES/README.md:36` </location>
<code_context>
background-attachment: fixed
opacity: 0.5
- filter: brightness(60%)
+ filter: brillo (60%)
features:
- - title: Online Almanac
</code_context>
<issue_to_address>
考虑保留“brightness”不翻译,以保持CSS属性的清晰度。
将“brightness”翻译为“brillo”可能会破坏CSS过滤属性,并在代码中使用时导致混淆。
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
filter: brillo (60%)
=======
filter: brightness(60%)
>>>>>>> REPLACE
</suggested_fix>
帮助我更有用!请点击 👍 或 👎 对每条评论进行评价,我将利用这些反馈来改进你的审查。
Original comment in English
Hey @Gzh0821 - I've reviewed your changes - here's some feedback:
- Don’t translate CSS properties or values (e.g., ‘filter: brillo(60%)’ is invalid CSS—keep the original ‘brightness(60%)’).
- The Russian
submitlevel.md
now contains English text instead of Russian localization—please restore the proper translated content. - Double-check the updated relative links in localized guides to ensure they still resolve to the correct pages.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Don’t translate CSS properties or values (e.g., ‘filter: brillo(60%)’ is invalid CSS—keep the original ‘brightness(60%)’).
- The Russian `submitlevel.md` now contains English text instead of Russian localization—please restore the proper translated content.
- Double-check the updated relative links in localized guides to ensure they still resolve to the correct pages.
## Individual Comments
### Comment 1
<location> `src/es-ES/README.md:36` </location>
<code_context>
background-attachment: fixed
opacity: 0.5
- filter: brightness(60%)
+ filter: brillo (60%)
features:
- - title: Online Almanac
</code_context>
<issue_to_address>
Consider keeping 'brightness' untranslated for CSS property clarity.
Translating 'brightness' to 'brillo' may break the CSS filter property and cause confusion if used in code.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
filter: brillo (60%)
=======
filter: brightness(60%)
>>>>>>> REPLACE
</suggested_fix>
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Sourcery 总结
应用来自 Crowdin 的西班牙语和俄语本地化更新至全站,包括修正的翻译和链接引用。
改进点:
杂项:
keybind-editor
和save-editor
组件的本地化 JSON 文件Original summary in English
Summary by Sourcery
Apply Crowdin-sourced localization updates for Spanish and Russian across the site, including corrected translations and link references
Enhancements:
Chores: