-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add SelectPopover component #4619
Conversation
- Display list of elements in a popover - Store selected element in state - Add callbacks to know when the selected element changed
This will allow us to set many options in there.
@@ -95,7 +95,7 @@ const SelectPopoverFormattedExample = createReactClass({ | |||
filterPlaceholder="Filter by colour" /> | |||
</div> | |||
|
|||
{selectedItem ? `You have selected ${selectedItem}` : 'Please select a colour!'} | |||
{selectedItem ? `You have selected ${selectedItem}` : 'Please select another colour!'} |
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.
colour -> color
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.
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.
😄
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.
LGTM!
This PR introduces a
SelectPopover
component that we want to use on actions and filters inTableList
components:Some features of this component: