-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
feat(wip): add Combobox component for timezone picker #6154
Conversation
Looking amazing! |
@m1yon I refactored the combo box so that it can be used in different components in the future, as well as cleaned up the change date component |
94ac85f
to
0be6e88
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work pretty well. I changed a few small things:
- Clicking outside of the dropdown/combobox close it
- Clicking the up/down arrows in the button now opens the combobox (did not before)
- Removed an unnecessary nested div.relative element
@alextran1502 @jrasm91 Thanks for picking this up and finishing it! I'm in the middle of a move so I haven't had a chance to work on it :/ Looks great though! |
No worries, hope the move goes well 😄 |
This change introduces a new Combobox component, a custom select with autocomplete functionality.
The purpose of this component is to replace the current timezone dropdown and be used wherever we need it in the future:
Current:
New (WIP):