Skip to content

fix(docs): remove duplicate install chrome from preview shell#155

Open
Gujiassh wants to merge 1 commit into
assistant-ui:mainfrom
Gujiassh:fix/preview-layout-minimal-152
Open

fix(docs): remove duplicate install chrome from preview shell#155
Gujiassh wants to merge 1 commit into
assistant-ui:mainfrom
Gujiassh:fix/preview-layout-minimal-152

Conversation

@Gujiassh

@Gujiassh Gujiassh commented Jun 7, 2026

Copy link
Copy Markdown

Summary

  • fixes Broken preview layout #152
  • remove the duplicate install command block from ComponentPreviewShell
  • keep install guidance in the docs header / sheet flow, which is the lighter touch requested in the issue
  • add a focused regression test that the preview shell no longer renders duplicate tool-agent / shadcn chrome above the preview area

Testing

  • pnpm --filter tool-ui-www lint:format
  • pnpm --filter tool-ui-www test -- apps/www/lib/tests/tool-ui/docs/component-preview-shell.test.tsx apps/www/lib/tests/tool-ui/docs/gallery-layout.test.ts apps/www/lib/tests/tool-ui/docs/install-snippet-analytics.test.ts

@rupic-app rupic-app Bot added stage/needs-review rupic stage: needs_review app/www rupic: changed package R0 rupic: review-status rank type/bugfix rupic: PR type labels Jun 25, 2026
@rupic-app rupic-app Bot mentioned this pull request Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app/www rupic: changed package R0 rupic: review-status rank stage/needs-review rupic stage: needs_review type/bugfix rupic: PR type

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken preview layout

1 participant