docs: add tiny-pro and tiny-editor in top nav menu#55
Conversation
WalkthroughThe changes add TinyPro product entries to the navigation menu configuration, including asset imports and metadata for menu items across playground and documentation sections, plus a demonstration site link in the product documentation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @packages/common/src/config/header.ts:
- Around line 157-163: The TinyEditor entry currently reuses the wrong logo
(tinyRobot); add a dedicated SVG file named tiny-editor.svg into
packages/common/src/assets/appIcon/, import it (e.g. import tinyEditor from
'.../tiny-editor.svg') and update the TinyEditor object in header.ts (the entry
with title 'TinyEditor') to set logo: tinyEditor and keep other fields the same
so the correct asset is used.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
packages/common/src/config/header.tspackages/opentiny-docs/src/views/vue-pro/start.md
🔇 Additional comments (4)
packages/opentiny-docs/src/views/vue-pro/start.md (1)
10-11: LGTM!The demo site URL is consistent with the playground entry added in
header.tsand provides a useful quick link for users to experience TinyPro.packages/common/src/config/header.ts (3)
6-6: LGTM!The SVG import follows the existing pattern for other product logos.
107-114: LGTM!The TinyPro playground entry is well-structured and consistent with the other items. The demo URL correctly matches the documentation in
start.md.
150-156: LGTM!The TinyPro documentation entry is properly configured with the correct logo and GitHub repository link.
| { | ||
| title: 'TinyEditor', | ||
| desc: '富文本编辑器', | ||
| href: 'https://opentiny.github.io/tiny-editor', | ||
| logo: tinyRobot, | ||
| github: 'https://github.com/opentiny/tiny-editor' | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, run the suggested check for editor-related SVG assets
fd -t f -e svg 'editor' packages/common/src/assets/Repository: opentiny/opentiny.design
Length of output: 50
🏁 Script executed:
# Check what imports tinyRobot and understand what it is
cd packages/common/src/config
head -50 header.ts | cat -nRepository: opentiny/opentiny.design
Length of output: 2085
🏁 Script executed:
# Search for where tinyRobot is imported from
rg 'tinyRobot' packages/common/src/config/header.ts -B 5 -A 2Repository: opentiny/opentiny.design
Length of output: 1309
🏁 Script executed:
# Check the assets directory structure to understand available logos
fd -t f -e svg '' packages/common/src/assets/ | head -20Repository: opentiny/opentiny.design
Length of output: 1021
Add dedicated logo asset for TinyEditor.
TinyEditor incorrectly uses tinyRobot logo, which is the asset for the TinyRobot project. Other products (TinyVue, TinyEngine, TinyPro, etc.) each have their own dedicated asset in packages/common/src/assets/appIcon/. Add a tiny-editor.svg asset for TinyEditor to maintain consistency.
🤖 Prompt for AI Agents
In @packages/common/src/config/header.ts around lines 157 - 163, The TinyEditor
entry currently reuses the wrong logo (tinyRobot); add a dedicated SVG file
named tiny-editor.svg into packages/common/src/assets/appIcon/, import it (e.g.
import tinyEditor from '.../tiny-editor.svg') and update the TinyEditor object
in header.ts (the entry with title 'TinyEditor') to set logo: tinyEditor and
keep other fields the same so the correct asset is used.
Summary by CodeRabbit
Release Notes
New Features
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.