-
Notifications
You must be signed in to change notification settings - Fork 2.4k
feat(ui): port welcome visuals from PR #8470 (no Cloud Agent) #9051
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Updated ChatTextArea border radius from rounded to rounded-lg - Updated TaskHeader padding and border radius styling - Modified RooHero to remove flex centering and add bounce animation - Updated RooTips with new content, Lucide icons, and improved layout - Modified chat.json translations for updated onboarding content - Updated HistoryPreview layout with header and improved spacing - Modified TaskItem styling with rounded-xl and font-light - Updated TaskItemFooter spacing - Changed history.json viewAllHistory text to "View all" These changes incorporate the visual improvements from PR #8470 without including any CloudAgents functionality or related cloud features.
Review Complete ✅Reviewed the latest commit adding missing localization keys. The changes are clean:
Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request. |
| "title": "可自訂模式", | ||
| "description": "具有專屬行為和指定模型的特定角色" | ||
| }, | ||
| "modelAgnostic": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this Traditional Chinese locale file, the modelAgnostic and docs keys are added but the values remain in English. Please localize these strings or document the reason they remain untranslated.
This comment was generated because it violated a code review rule: irule_C0ez7Rji6ANcGkkX.
| "description": "Personas especializadas com comportamentos próprios e modelos atribuídos" | ||
| }, | ||
| "modelAgnostic": { | ||
| "title": "Bring your own model", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The newly added keys modelAgnostic and docs in this Portuguese (Brazil) translation still use English text. Consider translating them to Portuguese for consistency.
| "title": "Bring your own model", | |
| "title": "Traga seu próprio modelo", |
This comment was generated because it violated a code review rule: irule_C0ez7Rji6ANcGkkX.
This PR copies visual changes from #8470 into the extension while explicitly excluding all Cloud Agent functionality.
Changes:
No Cloud API, CSP policy, or CloudAgents components were added. All UI tests pass:
cd webview-ui && npx vitest run => 94 passed, 0 failed.
Important
Ports welcome screen UI changes from PR #8470, excluding Cloud Agent functionality, with updated styles and tests.
ChatViewlayout to includeVersionIndicator,RooHero,RooTips, andHistoryPreviewfor authenticated or new users.Upsellshown only when logged out and after 4+ tasks;CloudAgentscomponent not rendered.ChatTextAreaandTaskHeaderupdated with rounded corners.HistoryPreviewandTaskItemhover/typography styles updated.ChatViewupsell/tips conditions,HistoryPreviewcontainer classes,TaskItemhover class.useTaskSearchand related state management fromChatView.tsx.This description was created by
for d2faee7. You can customize this summary. It will automatically update as commits are pushed.