Skip to content

Conversation

@brunobergher
Copy link
Collaborator

@brunobergher brunobergher commented Sep 16, 2025

Addresses more feedback about the new auto-approve UI:

  • That it was too wide in narrow setups
  • Some way to see the currently enabled toggles without clicking

Description

  • Makes 1 or 2-column layout dependent on available width for the extension
  • Includes the currently enabled auto-approved actions in the tooltip. This shows all enabled toggles, which is better than before, where while always visible, it got quickly truncated after 2 or 3 were enabled.

Test Procedure

  1. Open the auto-approve dropdown
  2. Resize the extension to quite wide and quite narrow

Screenshots / Videos

1
2
3

Important

Enhances the auto-approve UI in AutoApproveDropdown.tsx by making it responsive and updating tooltips, with corresponding updates in multiple language files.

  • UI Enhancements:
    • AutoApproveDropdown.tsx: Adjust layout to be responsive based on width, switching between 1 or 2 columns.
    • Update tooltip to show all enabled auto-approve actions, improving visibility over previous truncation.
  • Internationalization:
    • Update chat.json files across multiple languages (e.g., ca, de, en) to reflect new tooltip text and layout changes.
  • Miscellaneous:
    • Minor CSS changes to AutoApproveDropdown.tsx for width and layout adjustments.

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

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. UI/UX UI/UX related or focused labels Sep 16, 2025
@brunobergher brunobergher changed the title Bb/ux/responsive auto appove ux: Responsive Auto-Approve Sep 16, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Sep 16, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution! I've reviewed the changes and have some suggestions for improvement. Overall, this is a well-implemented responsive UI improvement that addresses the feedback mentioned in the PR description.

brunobergher and others added 6 commits September 17, 2025 15:03
Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 17, 2025
@mrubens mrubens merged commit 6ac6a92 into main Sep 17, 2025
9 checks passed
@mrubens mrubens deleted the bb/ux/responsive-auto-appove branch September 17, 2025 14:55
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Sep 17, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files. UI/UX UI/UX related or focused