-
Notifications
You must be signed in to change notification settings - Fork 592
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
Added Support for React Native Projects #230
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Lets just cleanup the readme a little bit and this will be good to go!
README.md
Outdated
@@ -108,6 +108,12 @@ A sequence is started via a call to `start()` on the `TapTargetSequence` instanc | |||
|
|||
For more examples of usage, please look at the included sample app. | |||
|
|||
## React Native |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets replace this section with
## Third Party Bindings
### React Native
Thanks to @prscX, you may now use this library with [React Native](https://github.com/facebook/react-native) via the module [here](https://github.com/prscX/react-native-taptargetview)
### Xamarin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @xiphirx for reviewing the request
I have made the requested changes. Please let me know in case any other changes required too
Thanks
Pranav
* Update README.md * Update README.md * Update README.md * Update README.md
* Add startWith and startAt to TapTargetSequence (KeepSafe#228) Previously it was cumbersome to restore state in a TapTargetSequence after rotation due to the fact that you needed to manually modify the list of targets you pass in to get back to your desired target. Now the class supports starting from a specified target id or position in the queue, allowing callers to keep a static list of targets for their sequence. * Added Support for React Native Projects (KeepSafe#230) * Update README.md * Update README.md * Update README.md * Update README.md * Add notion of isDismissing and gate onReady() callbacks on it (KeepSafe#243) * Add notion of isDismissing and gate onReady() callbacks on it There was a race condition I came across where a view could be animating out at the same time as the animation for dismissing the tap target. If the view animated out before the taptarget, it would trigger a layout again, and potentially redisplay the taptarget again. This gates onReady callbacks against the notion of "isDismissing". Tested locally in my project and confirmed this resolves the issue. * Return early (and earlier!) * Fix NPE when dismiss is called immediately (KeepSafe#244) Small "race" condition that shouldn't crash. Fixes KeepSafe#240 * 1.11.0 * Fix "sometimes" typo (KeepSafe#246) * Update Gradle version (KeepSafe#253) * Disable interaction during expand animation (KeepSafe#248) * Fix build (KeepSafe#256) * Update to latest support library + SDK version (KeepSafe#257) Was pretty outdated previously. * Use api configuration for public dependencies. (KeepSafe#258) Some support annotations and the v7 support Toolbar are both part of the public api, so they need to be specified as `api` in order for them to transitively picked up by downstream dependants. * Update build tools to most recent ones (KeepSafe#261) * update build tools to most recent ones * revert to last stable version and use gradle 4.5 * Use central build variables (KeepSafe#264) * in module "taptargetview" buildToolsVersion was missing und request an install of 26.0.2 * variable SupportLibraryVersion was just in one module * Handle dismiss before show * Replacement of "compile" with "implementation" (KeepSafe#291) As "compile" is going to be deprecated end-2018, this PR replaces with "implementation" that is the new usage. * Fix NPE from dismissal before determining center (KeepSafe#298) Fixes KeepSafe#279 * Fix not being able to cancel on last target (KeepSafe#299) Simplifies an early return + properly unsets currentView. Fixes KeepSafe#294 * Fix decor related issues (KeepSafe#300) This fixes issues relating to transparent nav bars, status bars and multi window mode. The library will now detect when the window it is attached to has a transparent system bar and properly adjust its clip bounds. Fixes KeepSafe#233 * Release 1.12.0 * Updated readme to reflect latest version
Hi @KeepSafeAccount
First of all I would like to appreciate for creating such a cool app tour 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 looking to use this library for React Native projects can easily do the same
Please let me know in case any changes are required
Thanks
Pranav