Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

SelectDropdown: Add support for value #446

Merged
merged 4 commits into from Jan 10, 2019
Merged

Conversation

ItsJonQ
Copy link
Contributor

@ItsJonQ ItsJonQ commented Jan 10, 2019

SelectDropdown: Add support for value

This update improves the SelectDropdown to be able to accept a
value prop. This adds API parity with Select as well as the
native HTML select.

value will be used to find the actual item from the original items prop.

Both selectedItem and value is supported for SelectDropdown.

An enhancement was made to Dropdown to add a new triggerStyle prop,
allowing you to specify inline styles for the Trigger wrapper
component.

This update has been E2E tested in the main application.

This update improves the `SelectDropdown` to be able to accept a
`value` prop. This adds API parity with `Select` as well as the
native HTML select.

`value` will be used to find the actual item from the original `items` prop.

Both `selectedItem` and `value` is supported for `SelectDropdown`.

An enhancement was made to `Dropdown` to add a new `triggerStyle` prop,
allowing you to specify inline styles for the Trigger wrapper
component.
@coveralls
Copy link

coveralls commented Jan 10, 2019

Pull Request Test Coverage Report for Build 1510

  • 24 of 24 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 1506: 0.0%
Covered Lines: 5735
Relevant Lines: 5735

💛 - Coveralls

@ItsJonQ ItsJonQ merged commit 34712f2 into master Jan 10, 2019
@ItsJonQ ItsJonQ deleted the select-dropdown-selected-value branch January 28, 2019 19:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants