Skip to content

Conversation

@brunobergher
Copy link
Collaborator

@brunobergher brunobergher commented Jun 17, 2025

Description

The core of this PR is changing the copy in the onboarding screen to emphasize Modes as a differentiator.
In the process, I cleaned up the UI a bit.
Translations included.

Test Procedure

  1. Run the extension with no existing configuration
  2. Look at the welcome screen

Type of Change

  • 🐛 Bug Fix: Non-breaking change that fixes an issue.
  • New Feature: Non-breaking change that adds functionality.
  • 💥 Breaking Change: Fix or feature that would cause existing functionality to not work as expected.
  • ♻️ Refactor: Code change that neither fixes a bug nor adds a feature.
  • 💅 Style: Changes that do not affect the meaning of the code (white-space, formatting, etc.).
  • 📚 Documentation: Updates to documentation files.
  • ⚙️ Build/CI: Changes to the build process or CI configuration.
  • 🧹 Chore: Other changes that don't modify src or test files.

Pre-Submission Checklist

  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Code Quality:
    • My code adheres to the project's style guidelines.
    • There are no new linting errors or warnings (npm run lint).
    • All debug code (e.g., console.log) has been removed.
  • Testing: doesn't apply imo
  • Branch Hygiene: My branch is up-to-date (rebased) with the main branch.
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Changeset: A changeset has been created using npm run changeset if this PR includes user-facing changes or dependency updates.
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

image

Important

This PR updates the onboarding UI to highlight Modes, improves UI styling, and updates translations across multiple languages.

  • UI Changes:
    • Updated RooHero component in RooHero.tsx to simplify the structure by removing unnecessary div wrapper.
    • Enhanced WelcomeView in WelcomeView.tsx with padding adjustments and font styling for better readability.
  • Translation Updates:
    • Updated welcome messages and instructions in multiple language files (welcome.json) to emphasize Modes and API key requirements.
    • Added incentives for using specific routers in translations.
  • Misc:
    • Removed redundant 'or' text in various language files to streamline the UI.

This description was created by Ellipsis for 885ef67ab9148f83a001a7465c2cc91faeada1ba. You can customize this summary. It will automatically update as commits are pushed.

@brunobergher brunobergher requested review from cte, jr and mrubens as code owners June 17, 2025 18:11
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. UI/UX UI/UX related or focused labels Jun 17, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jun 17, 2025
@hannesrudolph hannesrudolph added Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. PR - Needs Preliminary Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 17, 2025
@mrubens
Copy link
Collaborator

mrubens commented Jun 17, 2025

Nice! Only thing I noticed is that the hover state for the router buttons is a little intense, in my theme at least. Should we maybe make it a border?

Screenshot 2025-06-17 at 2 32 39 PM

@daniel-lxs daniel-lxs moved this from PR [Needs Prelim Review] to PR [Changes Requested] in Roo Code Roadmap Jun 19, 2025
Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

Looks great!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 20, 2025
@mrubens mrubens merged commit 886a8a7 into RooCodeInc:main Jun 20, 2025
12 of 13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 20, 2025
@github-project-automation github-project-automation bot moved this from PR [Changes Requested] to Done in Roo Code Roadmap Jun 20, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Changes Requested size:L This PR changes 100-499 lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants