Skip to content
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

Make list of modes multi-line on the prompts page #732

Merged
merged 1 commit into from
Feb 2, 2025
Merged

Conversation

mrubens
Copy link
Collaborator

@mrubens mrubens commented Feb 2, 2025

Need to rethink the design of this whole page, but IMO this is better for now.

Before:
Screenshot 2025-02-02 at 2 30 57 AM

After:
Screenshot 2025-02-02 at 2 33 30 AM


Important

Make the list of modes multi-line in PromptsView.tsx by adjusting flexbox properties for better readability.

  • UI Changes:
    • In PromptsView.tsx, change gap from 16px to 8px and flexWrap from nowrap to wrap for mode and support prompt lists.
    • Remove overflowX: auto, paddingBottom, and paddingRight from mode and support prompt lists.
    • Add padding: "4px 0" to mode and support prompt lists for better spacing.

This description was created by Ellipsis for a01d923. It will automatically update as commits are pushed.

Copy link

changeset-bot bot commented Feb 2, 2025

🦋 Changeset detected

Latest commit: a01d923

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@@ -472,13 +472,11 @@ const PromptsView = ({ onDone }: PromptsViewProps) => {
<div
Copy link

Choose a reason for hiding this comment

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

Consider extracting the repeated inline styles into a shared utility or a CSS class to reduce redundancy and improve maintainability. This is from our Development Standards: https://www.notion.so/Development-Standards-59febcf8ead647fd9c2ec3f60c22f3df?pvs=4#11869ad2d5818094a05ef707e188c0d5

@mrubens mrubens merged commit cddd92b into main Feb 2, 2025
9 checks passed
@mrubens mrubens deleted the fix_mode_wrapping branch February 2, 2025 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants