Skip to content

Conversation

skvale
Copy link
Contributor

@skvale skvale commented Apr 3, 2020

Description

Add an onSelect callback event for ComboboxInput, Select, and SearchInput and reflect the changes in the ComboboxInput and Select components.

[BREAKING]

  • ComboboxInput no longer takes a list prop, instead, it takes an options array of shape {key, text}
  • Select no longer takes children, instead, it takes an options array of shape {key, text}

fixes #921

@skvale skvale self-assigned this Apr 3, 2020
>
<div
className="fd-select__control"
className="fd-select__control is-success"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This snapshot is changed because the previous test misspelled validationState as uvalidationState

@netlify
Copy link

netlify bot commented Apr 3, 2020

Deploy preview for fundamental-react ready!

Built with commit ef852fa

https://deploy-preview-958--fundamental-react.netlify.com

Copy link
Contributor

@jbadan jbadan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 🥇

Add punctuation
@skvale skvale changed the title feat: add control onSelect callbacks feat: add onSelect callback to ComboboxInput;Select;SearchInput Apr 3, 2020
@skvale skvale merged commit 21b26cb into master Apr 3, 2020
@skvale skvale deleted the feat/add-control-callbacks branch April 3, 2020 15:37
@nickbaek
Copy link

nickbaek commented May 6, 2020

Hi team, I just wonder how can we retrieve the value that we have selected?

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.

Select component doesn't update the option on click
4 participants