Skip to content

Conversation

wujessica
Copy link
Contributor

@wujessica wujessica commented Jun 18, 2024

Description:

Closes OS-2175

This PR updates the settings drawer to reflect the latest designs. It adds toggle cards and separates tools by availability, ports over the change where deselecting the search_file tool will deselect files in the composer, and re-organizes and renames components.

Screen.Recording.2024-06-19.at.1.42.34.PM.mov

If there are errors and the tool is unavailable the toggle is hidden + error is shown:

Screenshot 2024-06-18 at 4 30 52 PM

AI Description

This PR introduces a new SettingsDrawer component and makes corresponding changes to replace the ConfigurationDrawer component.

  • The ConfigurationDrawer component and its imports have been removed.
  • The SettingsDrawer component has been added, which includes tabs for tools, files, and settings.
  • The FilesTab and ToolsTab components have been added to display file and tool information.
  • The Files and Tools components have been removed, and their functionality integrated into the new tabs.
  • The Settings component has been renamed to SettingsTab.
  • The Switch component has been updated to include a Field wrapper and style changes.
  • The ToggleCard component has been added to display tool information with a toggle.
  • Constants and references have been updated to reflect the new settings drawer.

Copy link
Contributor

@misspia-cohere misspia-cohere left a comment

Choose a reason for hiding this comment

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

the calculator description tho haha

@wujessica wujessica force-pushed the jw/new-composer-changes-pt-3 branch from f8ab403 to 86a9061 Compare June 21, 2024 15:27
@codecov-commenter
Copy link

codecov-commenter commented Jun 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@9184edf). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage        ?   86.66%           
=======================================
  Files           ?      149           
  Lines           ?     5549           
  Branches        ?        0           
=======================================
  Hits            ?     4809           
  Misses          ?      740           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@wujessica wujessica merged commit 0bf8b73 into main Jun 22, 2024
@wujessica wujessica deleted the jw/new-composer-changes-pt-3 branch June 22, 2024 18:56
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.

4 participants