Skip to content

Conversation

@matyasfodor
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jan 23, 2018

Codecov Report

Merging #835 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #835   +/-   ##
=======================================
  Coverage   53.03%   53.03%           
=======================================
  Files          29       29           
  Lines         511      511           
  Branches      109      109           
=======================================
  Hits          271      271           
  Misses        184      184           
  Partials       56       56

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18826c3...14916bb. Read the comment docs.

Copy link
Collaborator

@xavier-villelegier xavier-villelegier left a comment

Choose a reason for hiding this comment

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

Thanks @matyasfodor ! 👌

@xavier-villelegier xavier-villelegier changed the base branch from master to next January 23, 2018 17:30
@xavier-villelegier xavier-villelegier merged commit aa187a4 into react-native-elements:next Jan 23, 2018
xavier-villelegier pushed a commit that referenced this pull request Jan 28, 2018
* Update README.md - fixing typos (#835)

* Pass activeOpacity to TouchableOpacity inside Tile (#839)

* Pass activeOpacity to TouchableOpacity inside Tile

* Update Tile tests

* Comment `LinearGradient` for now (FIXME)

* feat(example): Cleanup remains from example app split

* Fix ListItem (#848)

Fix error on empty rightTitle prop by adding !!

* Separate checkbox single and checkbox with text styling (#850)

* (theming) Unify components under one primary color (#849)

* feat(theming): Remove not-used colors

* feat(theme): Add primary color and use through components

Added in button, button group, badge, and header

* Update snapshots

* Update underColor on buttonGroup to match selectedColor

* Snapshots

* Add color for Checkbox

* fix(Checkbox): Shifting width between checked and unchecked

* chore(packages): Update versions to fix warnings in jest

* added listItem right icon size so that it takes rightIcon size, or 28 (#852)

* uses thumbStyle.transform values in the thumb View (#820)

* [ts] Correct style type for IconObject (#851)

* Add ButtonComponent props and linear gradient props (#857)

* Add ButtonComponent props

* Use `linearGradientProps` + add error on mounting

* feat(button-group): Adds selectMultiple feature (#858)

Closes #713

* Platform specific SearchBar (#837) 🚀

* Add SearchBar wrapper

* Split SearchBar iOS and Android into separate files

* Add back default SearchBar

* Update root import

* Add back Input right/left icon

* Rename loadingIcon to loadingProps

* Specify `platform` PropType

* Add a fallback

* Move tests to `searchbar`

* Remove old Search component

* Fix loading props spreading overriding whole style

* Remove useless things

* Add tests for SearchBar

* Generate snapshots

* Add more tests

* Update tests

* Update documentation

* Fix typo

* Slightly changed the style of SearchBars

* Bump version to `1.0.0-beta2`

* Update snapshots from Searchbar changes

* feat(button): Re-add custom TouchableComponent

* feat(Button): Cleanup button code, update ts definitions (#859)

* Update SearchBars PropTypes

* feat(searchbar-both) Add additional defaultProps

Made tests pass

* Fix PropTypes warnings

* Update snapshots from searchbar android change

* feat(ts): Add definitons for searchbar (#860)

* Fix attributes spreading in platform specific searchbars

* feat(Button) Fix activeOpacity props

* fixed PricingCard to use new Button props

* test(jest): Update PricingCard snapshots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants