-
Notifications
You must be signed in to change notification settings - Fork 455
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
Add disable animation flag #25
Add disable animation flag #25
Conversation
Added disableAnimation flag to stop showing animation on tooltip
Updated code for more stability
disposeOnTap == null | ||
? true | ||
: (onTargetClick == null ? false : true), | ||
"onTargetClick is required if you're using disposeOnTap"), |
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.
Not sure why this is changed
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.
Is it due to code format.
@@ -58,3 +58,4 @@ build/ | |||
instapk.log* | |||
ios/Frameworks/ | |||
ios/Runner.xcworkspace/xcshareddata/ | |||
example/ios/Flutter/flutter_export_environment.sh |
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.
i think no need to update git ignore
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.
This file contains path information for resources required to project (Path can be very based on computer) which is not required in git.
Added disableAnimation flag to stop showing animation on tooltip
Updated code for more stability
Add disableanimation flag #22