Fix AI assistant demo panel placement#459
Conversation
|
Lost in the diff? Review this PR in Change Stack to follow the change map from intent to exact ranges. Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughHidden review stack artifactWalkthroughThe PR updates the AI assistant demo panel to use fixed overlay positioning that keeps it centered and within the viewport, replacing the previous absolute positioning. A changeset records this as a patch-level fix ensuring the panel remains visible. ChangesAI Assistant Panel Fixed Positioning
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
Fixes the generated React AI add-on header assistant so the opened chat panel stays inside the viewport. The panel previously opened with
left-fullfrom the header button and a fixed700pxwidth, which could push the input and send button off-screen.This changes the panel to a fixed, viewport-bound overlay with responsive width and height. It also adds a patch changeset for
@tanstack/create.Validation
pnpm --filter @tanstack/create buildpnpm --filter @tanstack/create testpnpm --filter @tanstack/create exec prettier --check src/frameworks/react/add-ons/ai/assets/src/components/demo-AIAssistant.tsxnx run-many --target=build --parallel 5pnpm test:unit && pnpm test:e2eSummary by CodeRabbit
Bug Fixes
Style