oblador / react-native-collapsible

Animated collapsible component for React Native, good for accordions, toggles etc
MIT License
2.46k stars 452 forks source link

Add style prop to create cards/containers #247

Closed M-Jas closed 6 years ago

M-Jas commented 6 years ago

What does this PR do?

Anything Else We Should Know?

Images

simulator screen shot - iphone x - 2018-10-03 at 10 07 25

iRoachie commented 6 years ago

Man that is a beautiful app 💯, seems like a duplicate of #246 . However if you can finish the changes before, then i'll merge yours.

Still need to add typescript definitions and fix the ci

M-Jas commented 6 years ago

@iRoachie Sorry I added the wrong picture, but the new one is up with the Accordion in full force. I'll take a look into #246 and Mr. Travis.

M-Jas commented 6 years ago

@iRoachie how do you run the ci locally? I don't have this '.' it speaks of.

iRoachie commented 6 years ago

You can run npm run test --fix and see if that works

M-Jas commented 6 years ago

@iRoachie helps if you npm i 😞. I think this last push should do the trick.

iRoachie commented 6 years ago

Lastly can you add it to defaultProps https://github.com/oblador/react-native-collapsible/blob/b21aa79e565f0478c243d0ec347df39bc48a55d9/Accordion.js#L31

iRoachie commented 6 years ago

Oops forgot the docs! Add it to the readme

M-Jas commented 6 years ago

Ok this time for real! Sorry for the back and forth.

iRoachie commented 6 years ago

Thanks mate! Release will be out soon