Skip to content
This repository has been archived by the owner on Jan 14, 2022. It is now read-only.

Added Support for React Native Projects #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

prscX
Copy link

@prscX prscX commented Feb 10, 2018

Hi @chase-calm

First of all I would like to appreciate for creating such a nice, cool library

I have created a React Native bridge plugin for using this library with React Native projects

I have added the same in README. Can you please merge this request so that if someone is looking to use this library for React Native projects can easily do the same

Please let me know in case any changes are required

react-native-tooltips

Thanks
Pranav

@prscX
Copy link
Author

prscX commented Mar 2, 2018

Hi @chase-calm, I have also changed delegate from weak type to strong:

@property (nonatomic, weak) id<SexyTooltipDelegate> delegate;

TO

@property (nonatomic, strong) id<SexyTooltipDelegate> delegate;

Reason: I have tried registering delegate for callback's but NSTimer somehow was removing the registered delegate.

Can you please merge the raise PR. Please let me know in case any discussion is required for the same.

Thanks
Pranav

@prscX
Copy link
Author

prscX commented Mar 21, 2018

Hi @tsheaff, Can you please merge this PR.

Thanks
Pranav

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant