Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Nov 6, 2025

Here's the non‑technical summary of what changed:

  • Export moved to the right of the mode dropdown; Import stays in the top toolbar. (Closes: [ENHANCEMENT] Move Import/Export to Modes view toolbar (next to "Create new mode") #8686)
  • Mode actions are next to the dropdown in this order: +, Edit, Delete, Export (with consistent spacing).
  • The separate API Configuration section was removed from this screen.
  • The old name field was removed; renaming now happens inline from the Edit icon.
  • After creating a mode, it's auto‑selected.
  • After deleting a mode, it switches back to the default mode.
  • Search now finds a mode immediately after you rename it.

No commits or pushes have been made.

Before After
image image

Important

Move Import/Export buttons to the toolbar in ModesView.tsx, maintaining existing functionality and removing them from the bottom section.

  • UI Changes:
    • Move Import/Export buttons to the toolbar in ModesView.tsx, next to the "Create new mode" button.
    • Remove Import/Export buttons from the bottom section in ModesView.tsx.
  • Functionality:
    • Retain existing behavior for Import/Export actions using vscode.postMessage.
    • Ensure buttons are keyboard-focusable and match toolbar style.
  • Misc:
    • No new strings added; i18n remains unchanged.

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

@roomote
Copy link

roomote bot commented Nov 6, 2025

Rooviewer Clock   See task on Roo Cloud

Re-review completed. The latest commit adds a useEffect hook to reset the import level to "project" each time the import dialog opens. This ensures consistent default behavior and works correctly with the existing controlled radio button components.

No issues found.

Previous reviews

Mention @roomote to ask your PR Fixer agent to address the feedback.

@dosubot dosubot bot added the UI/UX UI/UX related or focused label Nov 6, 2025
@hannesrudolph hannesrudolph moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Nov 6, 2025
@hannesrudolph hannesrudolph moved this from PR [Needs Prelim Review] to PR [Draft / In Progress] in Roo Code Roadmap Nov 6, 2025
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Nov 6, 2025
…ooCodeInc#9003

Removes import auto-select logic introduced in this PR:
- Removed prevModesForImportRef and snapshot-based detection
- Removed customModesRef used only for import logic
- Removed auto-select behavior in importModeResult handler

PR RooCodeInc#9003 implements a better backend-driven approach for import auto-select.
This change prevents merge conflicts; we keep create/delete auto-select which is unique to our PR.
@hannesrudolph hannesrudolph moved this from PR [Draft / In Progress] to PR [Needs Prelim Review] in Roo Code Roadmap Nov 6, 2025
… undefined for built-ins; controlled import level state; delegate delete fallback to backend
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR - Needs Preliminary Review size:XXL This PR changes 1000+ lines, ignoring generated files. UI/UX UI/UX related or focused

Projects

Status: PR [Needs Prelim Review]

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] Move Import/Export to Modes view toolbar (next to "Create new mode")

1 participant