fix: Modify tiny-robot official website#130
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThis PR replaces removed tiny-robot home sections with new sections, refactors existing sections into data-driven cards, updates the hero and technical content, and adds page-level reveal handling plus responsive layout changes. ChangesTiny-robot home page sections
Estimated code review effort: 4 (Complex) | ~60 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ 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: 4
🧹 Nitpick comments (2)
packages/home/src/views/tiny-robot-home/UseUs.vue (1)
20-25: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueUnused
TinyTagimport.
TinyTagis imported but never referenced in the template.🧹 Proposed fix
<script setup> -import { TinyTag } from '`@opentiny/vue`' import icon1 from '`@/assets/images/home/tiny-robot-home/use-us-icon1.png`'🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/home/src/views/tiny-robot-home/UseUs.vue` around lines 20 - 25, The UseUs.vue script has an unused TinyTag import that is never referenced in the component template. Remove the TinyTag import from the <script setup> block, keeping the remaining asset imports intact, so the component only imports symbols actually used in the file.packages/home/src/views/tiny-robot-home/FirstScreen.vue (1)
13-14: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueRemove now-dead
gotoChatcode.The "立即体验" button is commented out (line 13) and
gotoChat(lines 54-56) is no longer referenced anywhere, leaving dead code.♻️ Suggested cleanup
<div class="first-screen-content-footer"> - <!-- <tiny-button type="primary" round `@click`="gotoChat">立即体验</tiny-button> --> <tiny-button type="primary" round `@click`="gotoDocs">组件文档</tiny-button> </div>-const gotoChat = () => { - window.open('https://chat.opentiny.design/') -} - const gotoDocs = () => {Also applies to: 54-56
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/home/src/views/tiny-robot-home/FirstScreen.vue` around lines 13 - 14, The commented-out “立即体验” button and the unused gotoChat method are dead code in FirstScreen.vue, so remove both to keep the component clean. Delete the commented template entry and remove the gotoChat function from the script block, ensuring only the remaining gotoDocs flow is kept and no other references to gotoChat remain.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/home/src/views/tiny-robot-home/Cli.vue`:
- Around line 11-14: The “复制” control in Cli.vue is currently dead because
.copy-right only renders the icon/text and never triggers copying. Wire the
element to an existing click handler that uses useCopy() from
packages/home/src/tools/utils.js, and pass info.command so it copies the command
text to the clipboard and updates the copy/copy-success state. Reference the
Cli.vue component’s copy-right block and the useCopy helper to implement the
interaction without adding a new clipboard flow.
In `@packages/home/src/views/tiny-robot-home/FirstScreen.vue`:
- Line 125: The mobile banner background in FirstScreen.vue is pointing to the
wrong asset path and will fail to load. Update the mobile media query to use the
tiny-robot-home version of the banner asset, making sure the background-image
for the mobile state references the correct file under the tiny-robot-home
folder.
In `@packages/home/src/views/tiny-robot-home/RobotComponent.vue`:
- Line 12: The TinyTag usage in RobotComponent currently binds the label with
the value prop, but this component renders the visible text from text or default
slot content instead. Update the TinyTag instance in RobotComponent to pass
item.tag via the text prop or as slot content, using the TinyTag component as
the reference point so the label actually appears in the UI.
In `@packages/home/src/views/tiny-robot-home/UseUs.vue`:
- Around line 58-60: The external link handler in handleClickCase currently
opens URLs without tabnabbing protection. Update the window.open call used for
these third-party links to include the noopener and noreferrer features,
following the existing external-link convention used elsewhere in the codebase.
Keep the change localized to UseUs.vue and ensure all menu clicks that open
external destinations use the protected window.open signature.
---
Nitpick comments:
In `@packages/home/src/views/tiny-robot-home/FirstScreen.vue`:
- Around line 13-14: The commented-out “立即体验” button and the unused gotoChat
method are dead code in FirstScreen.vue, so remove both to keep the component
clean. Delete the commented template entry and remove the gotoChat function from
the script block, ensuring only the remaining gotoDocs flow is kept and no other
references to gotoChat remain.
In `@packages/home/src/views/tiny-robot-home/UseUs.vue`:
- Around line 20-25: The UseUs.vue script has an unused TinyTag import that is
never referenced in the component template. Remove the TinyTag import from the
<script setup> block, keeping the remaining asset imports intact, so the
component only imports symbols actually used in the file.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 1aad845b-6807-41b4-899e-bb968cb2af1f
⛔ Files ignored due to path filters (59)
packages/home/src/assets/images/home/tiny-robot-home/cli-bg.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/cli-icon.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-bubble.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-history.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-mcp.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-popover.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-sender.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/component-welcome.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/copy.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/dual-engine-icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/dual-engine-icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/dual-engine-icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-bg.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-chat.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-cli.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-component.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-kit.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-skill.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/scene-cases-icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/scene-cases-icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/scene-cases-icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/sender.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/technical_core_icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/technical_core_icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/technical_core_icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/technical_core_icon4.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/top.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/top_banner.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/top_mobile_banner.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/use-us-icon1.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tiny-robot-home/use-us-icon2.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tiny-robot-home/use-us-icon3.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tiny-robot-home/use-us-icon4.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tinyrobot_component1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_component2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_devops.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tinyrobot_experience.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_experience_icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_experience_icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_experience_icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_office.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tinyrobot_operations.pngis excluded by!**/*.pngpackages/home/src/assets/images/home/tinyrobot_scene_cases_img1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_scene_cases_img2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_scene_cases_img3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_scene_cases_img4.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_scene_cases_img5.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_scene_cases_img6.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_service_scene_bg.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_service_scene_icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_service_scene_icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_service_scene_icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_service_scene_icon4.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_technical_core_icon1.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_technical_core_icon2.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_technical_core_icon3.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_technical_core_icon4.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_top_banner.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tinyrobot_workforce.pngis excluded by!**/*.png
📒 Files selected for processing (16)
packages/home/src/assets/images/home/tiny-robot-home/dual-engine copy.webppackages/home/src/assets/images/home/tiny-robot-home/dual-engine.webppackages/home/src/assets/images/home/tiny-robot-home/scene-cases-img1.webppackages/home/src/assets/images/home/tiny-robot-home/scene-cases-img2.webppackages/home/src/assets/images/home/tiny-robot-home/scene-cases-img3.webppackages/home/src/views/tiny-robot-home/Cli.vuepackages/home/src/views/tiny-robot-home/DualEngine.vuepackages/home/src/views/tiny-robot-home/FirstScreen.vuepackages/home/src/views/tiny-robot-home/IntelligentExperience.vuepackages/home/src/views/tiny-robot-home/ProductMatrix.vuepackages/home/src/views/tiny-robot-home/RobotComponent.vuepackages/home/src/views/tiny-robot-home/SceneCases.vuepackages/home/src/views/tiny-robot-home/ServiceScene.vuepackages/home/src/views/tiny-robot-home/TechnicalCore.vuepackages/home/src/views/tiny-robot-home/UseUs.vuepackages/home/src/views/tiny-robot-home/index.vue
💤 Files with no reviewable changes (2)
- packages/home/src/views/tiny-robot-home/ServiceScene.vue
- packages/home/src/views/tiny-robot-home/IntelligentExperience.vue
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
packages/home/src/views/tiny-robot-home/index.vue (1)
83-92: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick winConsider respecting
prefers-reduced-motion.The fade/translate animation runs unconditionally for all users. Users with motion sensitivity preferences aren't accounted for.
♿ Proposed fix
:deep(.fade-in-up) { opacity: 0; transform: translateY(30px); transition: opacity 0.6s ease-out, transform 0.6s ease-out; } + + `@media` (prefers-reduced-motion: reduce) { + :deep(.fade-in-up) { + opacity: 1; + transform: none; + transition: none; + } + }🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/home/src/views/tiny-robot-home/index.vue` around lines 83 - 92, The fade-in animation in the tiny-robot-home view runs for everyone, including users who prefer reduced motion. Update the stylesheet for the .fade-in-up and .fade-in-up.is-visible rules to respect prefers-reduced-motion by disabling or simplifying the opacity/translate transition when that media query is active. Keep the fix localized to the existing scoped styles in the index.vue component.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/home/src/views/tiny-robot-home/index.vue`:
- Around line 27-38: The homepage fade-in behavior in the tiny-robot-home view
hides content by default and only reveals it through JavaScript, so a no-JS
fallback is needed. Update the `initFadeInUp`/`.fade-in-up` flow so content is
visible without script execution, ideally by making elements visible by default
and applying animation only after JS adds a state class, or by adding a CSS
fallback for the `fade-in-up`/`is-visible` classes. Keep the existing
`IntersectionObserver` path in `index.vue` but ensure section titles and content
remain readable if `initFadeInUp` never runs.
---
Nitpick comments:
In `@packages/home/src/views/tiny-robot-home/index.vue`:
- Around line 83-92: The fade-in animation in the tiny-robot-home view runs for
everyone, including users who prefer reduced motion. Update the stylesheet for
the .fade-in-up and .fade-in-up.is-visible rules to respect
prefers-reduced-motion by disabling or simplifying the opacity/translate
transition when that media query is active. Keep the fix localized to the
existing scoped styles in the index.vue component.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9838ed8f-3501-4551-9184-42ca3651a326
⛔ Files ignored due to path filters (5)
packages/home/src/assets/images/home/tiny-robot-home/product-chat.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-cli.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-component.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-kit.svgis excluded by!**/*.svgpackages/home/src/assets/images/home/tiny-robot-home/product-skill.svgis excluded by!**/*.svg
📒 Files selected for processing (9)
packages/home/src/views/tiny-robot-home/components/Cli.vuepackages/home/src/views/tiny-robot-home/components/DualEngine.vuepackages/home/src/views/tiny-robot-home/components/FirstScreen.vuepackages/home/src/views/tiny-robot-home/components/ProductMatrix.vuepackages/home/src/views/tiny-robot-home/components/RobotComponent.vuepackages/home/src/views/tiny-robot-home/components/SceneCases.vuepackages/home/src/views/tiny-robot-home/components/TechnicalCore.vuepackages/home/src/views/tiny-robot-home/components/UseUs.vuepackages/home/src/views/tiny-robot-home/index.vue
💤 Files with no reviewable changes (1)
- packages/home/src/views/tiny-robot-home/components/Cli.vue
pc:

移动端:

Summary by CodeRabbit
Summary
New Features
Improvements