Skip to content
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

ListItem v1 (Android & iOS) #945

Merged
merged 100 commits into from
Apr 1, 2018
Merged

ListItem v1 (Android & iOS) #945

merged 100 commits into from
Apr 1, 2018

Conversation

martinezguillaume
Copy link
Collaborator

No description provided.

Monte Thakkar and others added 7 commits February 24, 2018 19:35
* Updated readme with RNW todo + opencollective

* Rename back `text` to `title`

* Fix tests and Princing card

* Rename `clearText` into `clear`

* Apply default props on iOS SearchBar

* Update SearchBar wrapper with methods

* Update docs and ts

* Fix android onClear naming in PropTypes

* Remove useless condition

* Use default props

* Update snapshots

* Add more tests

* Update cancel methods doc
martinezguillaume and others added 15 commits February 26, 2018 12:03
* Updated readme with RNW todo + opencollective

* Rename back `text` to `title`

* Fix tests and Princing card

* Rename `clearText` into `clear`

* Apply default props on iOS SearchBar

* Update SearchBar wrapper with methods

* Update docs and ts

* Fix android onClear naming in PropTypes

* Remove useless condition

* Use default props

* Update snapshots

* Add more tests

* Update cancel methods doc
* Header title align

* Update snapshots and formatting

* Rename centerComponentStyles to centerComponentStyle

* Add ts defs for new props

* Add docs for new header props

* Remove redundant android statusbar height

* docs: Allow viewing `next` docs in version switch

* docs(header): Add left aligned center component example
* Warning only for non-expo user

* Auto import LinearGradient on Expo

* Show warning if non-expo user

* Update example

* Update snapshot
* Fixes #947 Input is not exported in TypeScript definitions

* Use react-native TextInputProperties and match type description to docs description

* Add missing methods for Input component
@iRoachie iRoachie added this to the Beta 4 Release milestone Mar 1, 2018
* Fix styled-component for Input

* Fix styled-component for Button
@react-native-elements-ci
Copy link
Collaborator

react-native-elements-ci commented Mar 27, 2018

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.

Let's break everything 🔥

@Monte9
Copy link
Collaborator

Monte9 commented Apr 1, 2018

here we goo.. 🤞

@codecov
Copy link

codecov bot commented Apr 1, 2018

Codecov Report

Merging #945 into next will decrease coverage by 0.48%.
The diff coverage is 72%.

Impacted file tree graph

@@            Coverage Diff            @@
##             next    #945      +/-   ##
=========================================
- Coverage   56.38%   55.9%   -0.49%     
=========================================
  Files          34      33       -1     
  Lines         603     610       +7     
  Branches      108     110       +2     
=========================================
+ Hits          340     341       +1     
- Misses        211     215       +4     
- Partials       52      54       +2
Impacted Files Coverage Δ
src/input/Input.js 18.75% <ø> (ø) ⬆️
src/searchbar/SearchBar-ios.js 61.76% <ø> (ø) ⬆️
src/searchbar/SearchBar-android.js 59.37% <ø> (ø) ⬆️
src/divider/Divider.js 100% <100%> (ø) ⬆️
src/list/ListItem.js 73.07% <68.18%> (-19.24%) ⬇️

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 754dc25...b5e7dee. Read the comment docs.

@xavier-villelegier xavier-villelegier merged commit 92e5e83 into next Apr 1, 2018
@iRoachie iRoachie deleted the listitem-v1 branch April 2, 2018 15:32
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.

None yet

8 participants