Conversation
- Replace manual createPortal + click-outside in variations-button with shadcn Popover - Replace manual createPortal + click-outside in critique-mode-button with shadcn Popover - Replace manual dropdown with click-outside useEffect in toolbar with shadcn DropdownMenu - Replace inline SVGs with lucide-react icons (Settings, Menu) - Use shadcn Separator for toolbar dividers
- Delete ToolButton component, replace with shadcn Button ghost variant - Migrate compass.tsx and zoom.tsx to shadcn Button - Migrate prompt-bar.tsx raw buttons to shadcn Button - Migrate image-pill.tsx remove button to shadcn Button - Migrate add-media-button.tsx to shadcn Button - Migrate context-toolbar.tsx separator to shadcn Separator - Migrate pipeline-status.tsx progress bar to shadcn Progress - Migrate ai-prompt-input.tsx textarea to shadcn Textarea - Replace inline SVGs with lucide-react icons (Loader2, X, AlertTriangle)
Complete wave 2 migration across apps/web/src/: - **Settings**: general, provider-list, provider-config-form, provider-selector, settings-modal, settings-personalization - **Export**: export-menu - **Comments**: comment-input, comment-thread - **Feedback**: feedback-modal, bug-icon - **Mode Sidebar**: mode-buttons, preset-button, system-prompt-button - **Design**: summary-list, summary-dialog - **Routes**: index.tsx - **Widgets**: error-boundary, prompt-library, update-notification - **Context Toolbar**: remix-button Replaced: - ~42 raw <button> elements → shadcn <Button> - 8 inline SVGs → lucide-react icons - 5 raw <input> elements → shadcn <Input> - 3 raw <textarea> elements → shadcn <Textarea> - 2 raw separator <div> patterns → shadcn <Separator> Exclusions respected: - settings-sidebar nav buttons (semantic nav) - design-card comment pin (dynamic STATUS_COLORS) - @mantine/hooks useClickOutside Build: 11/11 successful ✅ Verification: All grep sweeps clean ✅
Convert named exports to default exports to support direct file imports in lazy-loaded components.
Replace barrel exports with direct file imports for comment components. Reorganize imports for clarity.
Use specific React types (Component, ComponentType, ErrorInfo, ReactNode, PropsWithChildren) instead of namespace-prefixed types.
Extend PropsWithChildren for container components. Add field-sizing:fixed for consistent textarea height and margin to footer.
Add whitespace-normal to allow text wrapping in popover options. Add size=sm to button for consistent sizing.
Remove unused jotai imports and add Textarea component import.
…in onboarding flow
- Restore welcome-modal.tsx from git history - Fresh start now shows WelcomeModal instead of SettingsDialog - Take tour: starts TutorialTour - Skip for now: shows canvas (no SettingsDialog) - Re-add showWelcomeAtom to onboarding atoms - Restore showWelcomeAtom tests
Reuse settings content in tour view without duplicating layout and section rendering logic.
Mark AI Provider, Add Provider, and Unsplash API Key with data-tour attributes for tour-kit step targeting.
Non-interactive replica at z-[45] with pointer-events-none for use during settings tour steps.
Add provider-setup, add-provider, and unsplash-key steps between toolbar and all-set. Wire currentTourStepIdAtom for step tracking.
Close real SettingsDialog during settings tour steps, render SettingsTourView replica instead. Support ?tour param for replay.
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
calca | 13749cf | May 07 2026, 01:41 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR brings together a significant set of improvements spanning UI modernization, onboarding enhancements, analytics integration, and release process updates.
Key highlights:
SettingsTourView, tour step tracking with Jotai atoms, and data-tour attributes for guided user flowsSettingsContentfromSettingsDialog, removed deprecatedSettingsModalcomponentvalidateavailable per-package via--cwdflag for better DXRelated Issues
Solves #16 , #4
Type of Change
Checklist
bun run validate).worktrees/)License Acknowledgment
By submitting this pull request, I confirm that:
packages/pro/)Additional Notes
Testing coverage: 5 commits on onboarding, analytics, and feedback components with integrated tests updated. Desktop auto-updater improved with
getLocalInfousage.Dependencies: bun.lock updated; all workspace packages synchronized to v0.5.0.
Breaking changes: Deprecated
SettingsModalremoved; replaced withSettingsDialogin route handlers.