Skip to content

feat: hide tiny-robot immediate experience entry#117

Merged
hexqi merged 1 commit into
devfrom
wyp/delete-play-0416
Apr 16, 2026
Merged

feat: hide tiny-robot immediate experience entry#117
hexqi merged 1 commit into
devfrom
wyp/delete-play-0416

Conversation

@wuyiping0628
Copy link
Copy Markdown
Contributor

@wuyiping0628 wuyiping0628 commented Apr 16, 2026

Summary by CodeRabbit

  • UI Changes
    • Removed direct access to chat feature from the home screen and service details page, leaving the documentation link as the primary call-to-action.

@wuyiping0628 wuyiping0628 changed the title feat: hide the immediate experience entry feat: hide tiny-robot immediate experience entry Apr 16, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

Walkthrough

Two Vue component templates had their primary "立即体验" (Try Now) buttons removed via commenting. These buttons previously provided direct user access to the chat feature from the home screen and service scene. No exported declarations or control flow logic were altered.

Changes

Cohort / File(s) Summary
Button UI Removal
packages/home/src/views/tiny-robot-home/FirstScreen.vue, ServiceScene.vue
Disabled the primary "立即体验" buttons (gotoChat handler) in both component templates by commenting out markup. Secondary action buttons remain functional. Associated styles remain but are no longer applied due to DOM absence.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Poem

🐰 A rabbit removed two buttons with care,
Chat pathways vanished into thin air,
Comments now shield what once was bright,
Tidy templates rest in the morning light! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: hiding the 'immediate experience' entry points in the tiny-robot home interface by commenting out primary action buttons in two components.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wyp/delete-play-0416

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (2)
packages/home/src/views/tiny-robot-home/FirstScreen.vue (1)

13-13: Apply the same cleanup here: delete commented button markup.

Line 13 hides the CTA as intended, but keeping commented code in the template is avoidable tech debt. Prefer removing the node entirely.

Suggested cleanup
-          <!-- <tiny-button type="primary" round `@click`="gotoChat">立即体验</tiny-button> -->
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/home/src/views/tiny-robot-home/FirstScreen.vue` at line 13, Remove
the commented CTA button node from the FirstScreen.vue template (the HTML
comment containing <tiny-button type="primary" round
`@click`="gotoChat">立即体验</tiny-button>) to avoid leaving dead/commented template
code; after deletion, run a quick search for gotoChat to confirm it's still
used—if not referenced elsewhere, remove the unused gotoChat method from the
component to clean up related dead code.
packages/home/src/views/tiny-robot-home/ServiceScene.vue (1)

31-33: Remove the commented CTA block instead of keeping dead template code.

Line 31–33 already disables the CTA, but leaving it commented in-template makes future maintenance noisier. Prefer deleting this block in the final diff (reintroduce via a real condition/flag if needed later).

Suggested cleanup
-              <!-- <div class="scene-section-button">
-                <tiny-button round `@click`="gotoChat">立即体验</tiny-button>
-              </div> -->
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/home/src/views/tiny-robot-home/ServiceScene.vue` around lines 31 -
33, Remove the dead commented CTA block in the ServiceScene.vue template (the
commented <div class="scene-section-button"> with <tiny-button round
`@click`="gotoChat">立即体验</tiny-button>) instead of keeping it in-template; delete
those commented lines and, if you intend to reintroduce a CTA later, implement
it behind a real condition/prop/computed (e.g., v-if or a feature flag) tied to
the component's data or methods like gotoChat so no commented code remains.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@packages/home/src/views/tiny-robot-home/FirstScreen.vue`:
- Line 13: Remove the commented CTA button node from the FirstScreen.vue
template (the HTML comment containing <tiny-button type="primary" round
`@click`="gotoChat">立即体验</tiny-button>) to avoid leaving dead/commented template
code; after deletion, run a quick search for gotoChat to confirm it's still
used—if not referenced elsewhere, remove the unused gotoChat method from the
component to clean up related dead code.

In `@packages/home/src/views/tiny-robot-home/ServiceScene.vue`:
- Around line 31-33: Remove the dead commented CTA block in the ServiceScene.vue
template (the commented <div class="scene-section-button"> with <tiny-button
round `@click`="gotoChat">立即体验</tiny-button>) instead of keeping it in-template;
delete those commented lines and, if you intend to reintroduce a CTA later,
implement it behind a real condition/prop/computed (e.g., v-if or a feature
flag) tied to the component's data or methods like gotoChat so no commented code
remains.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5dc598b6-ada2-4a28-a7ce-2b8d72bfc28c

📥 Commits

Reviewing files that changed from the base of the PR and between fc42b1b and dbd5cdb.

📒 Files selected for processing (2)
  • packages/home/src/views/tiny-robot-home/FirstScreen.vue
  • packages/home/src/views/tiny-robot-home/ServiceScene.vue

@hexqi hexqi merged commit c47fda9 into dev Apr 16, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants