Skip to content

fix: Correctly pass down classname props to Combobox component #3143

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

Merged

Conversation

Ivcota
Copy link
Contributor

@Ivcota Ivcota commented Apr 23, 2025

Summary

Related Issues or PRs

How To Test

  1. Run storybook
  2. Visit http://localhost:9009/?path=/docs/components-combo-box--docs
  3. See With Custom Input Class Names

Screenshots (optional)

CleanShot 2025-04-23 at 11 16 55

@Ivcota Ivcota requested review from a team as code owners April 23, 2025 18:19
@Ivcota Ivcota changed the title combobox: pass down input classnames fix: Correctly pass down classname props to Combobox component Apr 23, 2025
@brandonlenz brandonlenz enabled auto-merge (squash) May 15, 2025 14:35
@brandonlenz brandonlenz disabled auto-merge May 15, 2025 14:35
@brandonlenz
Copy link
Contributor

@Ivcota thanks for the contribution. Please update from main and this should merge. If it requires approval again, give me a ping!

@brandonlenz brandonlenz enabled auto-merge (squash) May 15, 2025 14:36
@brandonlenz
Copy link
Contributor

@all-contributors please add @Ivcota for bug, code

Copy link
Contributor

@brandonlenz

I've put up a pull request to add @Ivcota! 🎉

@brandonlenz brandonlenz merged commit 2086ab8 into trussworks:main Jun 3, 2025
6 checks passed
@brandonlenz
Copy link
Contributor

@all-contributors please add @Ivcota for bug, code

Copy link
Contributor

@brandonlenz

@Ivcota already contributed before to bug, code

@brandonlenz
Copy link
Contributor

@all-contributors please add @Ivcota for bug, code

Copy link
Contributor

@brandonlenz

I've put up a pull request to add @Ivcota! 🎉

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