-
Notifications
You must be signed in to change notification settings - Fork 336
feat:Synchronize release-3.28.0 to dev #3954
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
WalkthroughChangelog and release documentation updated for v3.28.0/v2.28.0 (date bumped to 2025-12-29), including Chinese localized changelog, a new release article, extensive new "What's Changed"/"Bug Fixes"/"Other Changes" entries, and updated contributor lists. Playground app version constants and version list were bumped to 3.28 with minor formatting edits. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
Pre-merge checks✅ Passed checks (3 passed)
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. Comment |
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.
Actionable comments posted: 1
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
examples/sites/demos/pc/webdoc/changelog-en.mdexamples/sites/demos/pc/webdoc/changelog.mdexamples/sites/playground/App.vue
🧰 Additional context used
🪛 LanguageTool
examples/sites/demos/pc/webdoc/changelog.md
[style] ~53-~53: Consider using a different verb for a more formal wording.
Context: ...com//pull/3813> - fix: fixed an issue where pipeline errors were cau...
(FIX_RESOLVE)
[style] ~56-~56: Consider using a different verb for a more formal wording.
Context: ...com//pull/3828> - fix: fixed an issue where icon-multicolor failed t...
(FIX_RESOLVE)
[grammar] ~79-~79: Ensure spelling is correct
Context: ... when deleting tabs in the mobile-first vue2 environment. by @discreted66 in <https:...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
examples/sites/demos/pc/webdoc/changelog-en.md
[style] ~53-~53: Consider using a different verb for a more formal wording.
Context: ...com//pull/3813> - fix: fixed an issue where pipeline errors were cau...
(FIX_RESOLVE)
[style] ~56-~56: Consider using a different verb for a more formal wording.
Context: ...com//pull/3828> - fix: fixed an issue where icon-multicolor failed t...
(FIX_RESOLVE)
[grammar] ~79-~79: Ensure spelling is correct
Context: ... when deleting tabs in the mobile-first vue2 environment. by @discreted66 in <https:...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[style] ~168-~168: Consider using a different verb for a more formal wording.
Context: ...com//pull/3698> - fix: fix the issue of missing the tiny prefix in...
(FIX_RESOLVE)
[style] ~178-~178: Consider using a different verb for a more formal wording.
Context: .../tiny-vue/pull/3742> - fix: [tree-menu] fix the issue of icons not being referenced...
(FIX_RESOLVE)
[style] ~193-~193: Consider using a different verb for a more formal wording.
Context: ...tiny/tiny-vue/pull/3779> - fix(drawer): Fixed the problem that the table component in...
(FIX_RESOLVE)
[grammar] ~203-~203: Ensure spelling is correct
Context: ...eric modelValue type should not include underfined by @wuyiping0628 in <https://github.com...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~219-~219: Use a hyphen to join words.
Context: ...w Features 🎉 - feat(steps): Add wizard style step bar itemStyle differentiated ...
(QB_NEW_EN_HYPHEN)
[style] ~243-~243: Using “back” with the verb “return” may be redundant.
Context: ...id): grid promise validate return value back to undefined by @gimmyhehe in <https://...
(RETURN_BACK)
[style] ~259-~259: Unless you want to emphasize “not”, use “cannot” which is more common.
Context: ...y/tiny-vue/pull/3641> - fix(grid): grid can not validate on active by @gimmyhehe in <ht...
(CAN_NOT_PREMIUM)
[style] ~276-~276: Unless you want to emphasize “not”, use “cannot” which is more common.
Context: ... fix(grid): fix use title function text can not overflow ellipsis by @gimmyhehe in <htt...
(CAN_NOT_PREMIUM)
🪛 markdownlint-cli2 (0.18.1)
examples/sites/demos/pc/webdoc/changelog-en.md
131-131: Multiple headings with the same content
(MD024, no-duplicate-heading)
206-206: Multiple headings with the same content
(MD024, no-duplicate-heading)
215-215: Multiple headings with the same content
(MD024, no-duplicate-heading)
291-291: Multiple headings with the same content
(MD024, no-duplicate-heading)
🔇 Additional comments (6)
examples/sites/playground/App.vue (4)
18-18: LGTM!Version constant correctly updated to reflect the 3.28 release.
29-29: LGTM!Version array correctly updated to include 3.28 while maintaining the three most recent versions pattern.
93-95: LGTM!Formatting change improves readability without altering functionality.
279-281: LGTM!Formatting change for better readability without affecting the generated URL.
examples/sites/demos/pc/webdoc/changelog.md (1)
7-126: LGTM!The v3.28.0/v2.28.0 changelog is comprehensive and well-structured, appropriately documenting new features, bug fixes, other changes, and contributors for this release.
examples/sites/demos/pc/webdoc/changelog-en.md (1)
7-295: LGTM!The changelog entries for v3.28.0 and previous versions are comprehensive and well-documented in English.
| # 更新日志 | ||
|
|
||
| The Tiny Vue team uses a monthly release strategy under normal circumstances. | ||
| Tiny Vue 团队在正常情况下使用 每月 发布策略。 | ||
|
|
||
| On this page, you can only see the log records of the latest three iterations of our changelog. If you want to see the full record, you can view: [Release](https://github.com/opentiny/tiny-vue/releases) | ||
| 在此页面上,您只能看到我们的 更新日志 最新三个迭代的日志记录,如您要查看完整记录可以查看:[Release](https://github.com/opentiny/tiny-vue/releases) |
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.
Localization inconsistency: Chinese header in English changelog.
The file is named changelog-en.md (indicating English version), but lines 1-5 contain Chinese text. This creates confusion for users expecting English documentation.
The header and introductory text should be translated to English:
- Line 1: "# 更新日志" → "# Changelog"
- Line 3: Translate the release strategy description
- Line 5: Translate the log viewing instructions
🔎 Proposed translation
-# 更新日志
+# Changelog
-Tiny Vue 团队在正常情况下使用 每月 发布策略。
+The Tiny Vue team follows a monthly release strategy under normal circumstances.
-在此页面上,您只能看到我们的 更新日志 最新三个迭代的日志记录,如您要查看完整记录可以查看:[Release](https://github.com/opentiny/tiny-vue/releases)
+On this page, you can only see the changelog for the latest three iterations. For the complete changelog, please visit: [Release](https://github.com/opentiny/tiny-vue/releases)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # 更新日志 | |
| The Tiny Vue team uses a monthly release strategy under normal circumstances. | |
| Tiny Vue 团队在正常情况下使用 每月 发布策略。 | |
| On this page, you can only see the log records of the latest three iterations of our changelog. If you want to see the full record, you can view: [Release](https://github.com/opentiny/tiny-vue/releases) | |
| 在此页面上,您只能看到我们的 更新日志 最新三个迭代的日志记录,如您要查看完整记录可以查看:[Release](https://github.com/opentiny/tiny-vue/releases) | |
| # Changelog | |
| The Tiny Vue team follows a monthly release strategy under normal circumstances. | |
| On this page, you can only see the changelog for the latest three iterations. For the complete changelog, please visit: [Release](https://github.com/opentiny/tiny-vue/releases) |
🤖 Prompt for AI Agents
In examples/sites/demos/pc/webdoc/changelog-en.md around lines 1 to 5, the
content is in Chinese but the file is the English changelog; replace the Chinese
header and sentences with English: change line 1 to "# Changelog", translate the
release strategy sentence on line 3 to an English equivalent (e.g., "The Tiny
Vue team normally follows a monthly release cadence."), and translate the log
viewing instructions on line 5 to English (e.g., "This page shows the changelog
for the latest three iterations; to view the full history, see:
https://github.com/opentiny/tiny-vue/releases"). Ensure punctuation and link
remain correct.
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.
Actionable comments posted: 2
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
examples/sites/demos/pc/webdoc/v3.28.0-release-article.md
🧰 Additional context used
🪛 LanguageTool
examples/sites/demos/pc/webdoc/v3.28.0-release-article.md
[uncategorized] ~223-~223: 动词的修饰一般为‘形容词(副词)+地+动词’。您的意思是否是:必要"地"加
Context: ...合理设置 isLeaf 属性** - 明确标识哪些节点是叶子节点,避免不必要的加载请求 2. 使用缓存机制 - 对已加载的节点进行缓存,避免重复请求 ...
(wb4)
[uncategorized] ~759-~759: 动词的修饰一般为‘形容词(副词)+地+动词’。您的意思是否是:灵活"地"部署
Context: ...** - 构建成功后自动触发 优势: - 更好的错误定位 - 更灵活的部署控制 - 更快的反馈循环 --- ## 👥 新贡献者 感谢以下新贡献...
(wb4)
[uncategorized] ~760-~760: 动词的修饰一般为‘形容词(副词)+地+动词’。您的意思是否是:快"地"反馈
Context: ...自动触发 优势: - 更好的错误定位 - 更灵活的部署控制 - 更快的反馈循环 --- ## 👥 新贡献者 感谢以下新贡献者的加入: - @...
(wb4)
🪛 markdownlint-cli2 (0.18.1)
examples/sites/demos/pc/webdoc/v3.28.0-release-article.md
30-30: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
81-81: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
707-707: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
713-713: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
719-719: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
801-801: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
810-810: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
821-821: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
838-838: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
849-849: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
|
|
||
| ![选择器组件对比图] | ||
|
|
||
| ``` |
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.
Specify language for fenced code block.
Line 30 has a code block without a language identifier. Add a language tag (e.g., text or plaintext) for proper syntax highlighting and accessibility.
-```
+```text
旧版本:select | tree-select | grid-select | base-select🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
30-30: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
In examples/sites/demos/pc/webdoc/v3.28.0-release-article.md around line 30, the
fenced code block is missing a language identifier; update the opening triple
backticks to include a language tag such as text (e.g., change ``` to ```text)
so the block becomes ```text and retain the code content on the following line
for proper syntax highlighting and accessibility.
| **3. 维护成本降低** | ||
| 现在只需要在一个地方(`select-wrapper`)修复 bug 或添加功能,所有选择器组件都能自动受益。 | ||
|
|
||
| **4. 性能优化** |
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.
Use proper markdown headings instead of bold emphasis for subsections.
Lines 81, 707, 713, 719, 801, 810, 821, 838, and 849 use bold text (e.g., **4. 性能优化**) to denote subsections. Convert these to proper markdown heading syntax (#### or ###) to maintain semantic structure, improve accessibility, and comply with markdown standards.
🔎 Proposed fixes for heading hierarchy
-**4. 性能优化**
+#### 4. 性能优化-**1. Vite 配置优化**
+#### 1. Vite 配置优化-**2. 构建脚本优化**
+#### 2. 构建脚本优化-**3. 依赖管理优化**
+#### 3. 依赖管理优化-**阶段一:新功能使用新组件**
+#### 阶段一:新功能使用新组件-**阶段二:逐步替换旧组件**
+#### 阶段二:逐步替换旧组件-**阶段三:全面迁移**
+#### 阶段三:全面迁移-**问题一:样式不匹配**
+#### 问题一:样式不匹配-**问题二:事件处理**
+#### 问题二:事件处理Also applies to: 707-707, 713-713, 719-719, 801-801, 810-810, 821-821, 838-838, 849-849
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
81-81: Emphasis used instead of a heading
(MD036, no-emphasis-as-heading)
PR
feat:同步3.28.0版本代码到主干分支
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.