Skip to content

RN upgrade changes to Overlay.ios.js#51

Open
aakashsigdel wants to merge 6 commits into
brentvatne:masterfrom
aakashsigdel:master
Open

RN upgrade changes to Overlay.ios.js#51
aakashsigdel wants to merge 6 commits into
brentvatne:masterfrom
aakashsigdel:master

Conversation

@aakashsigdel

Copy link
Copy Markdown

Make it compatible with RN 0.28

@theopolisme

Copy link
Copy Markdown

+1

@vinch

vinch commented Jul 20, 2016

Copy link
Copy Markdown

+1

@skv-headless

Copy link
Copy Markdown

@brentvatne I confirm. Please merge it. @aakashsigdel please rebase pr.

@aakashsigdel

Copy link
Copy Markdown
Author

@skv-headless done

@skv-headless

Copy link
Copy Markdown

Could you return previous indentations. If you don't like them maybe better to create separate pr?

@aakashsigdel

aakashsigdel commented Aug 12, 2016

Copy link
Copy Markdown
Author

@skv-headless if you look at the repo, the file is already indented to 4 spaces. I think it was due to some other commit. Do you still want it to be 2 spaces?
https://github.com/brentvatne/react-native-overlay/blob/master/Overlay.ios.js

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.

5 participants