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

🎨 style: parameters panel update #4780

Merged
merged 3 commits into from
Nov 23, 2024
Merged

🎨 style: parameters panel update #4780

merged 3 commits into from
Nov 23, 2024

Conversation

berry-13
Copy link
Collaborator

Summary

This pull request includes several changes to the user interface and code quality. The most important changes involve replacing the doubleClickHandler prop with onDoubleClick for various components, removing unused imports, and enhancing the DynamicInput component to handle different input types, along with some Dynamic* style changes

Before After
image image

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • Style update

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes

…s into a single DynamicInput component; fix: UI crashing when typing a character instead of number in max context/output tokens
@berry-13 berry-13 changed the title style: UI update/fixes style: parameters panel update Nov 22, 2024
@berry-13 berry-13 changed the title style: parameters panel update 🎨 style: parameters panel update Nov 22, 2024
@danny-avila danny-avila merged commit ead9e11 into main Nov 23, 2024
2 checks passed
@danny-avila danny-avila deleted the ui-update branch November 23, 2024 00:10
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