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

CI RUN for pull/5844 #5980

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

CI RUN for pull/5844 #5980

wants to merge 3 commits into from

Conversation

abeatrix
Copy link
Contributor

Force CI run for #5844

Test plan

See #5844

Changelog

- Add max height and vertical scrolling to model selector popover
- Improve accessibility for long lists of model options
- Utilize tw-max-h-[80vh] and tw-overflow-y-auto Tailwind classes
- Removes redundant `cn` utility function call in `ModelSelectField` component
- Updates the `className` prop on the `CommandList` component to use a static string instead

## Changelog

No behavioral changes, only a minor code cleanup.
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