Skip to content
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 guards ( Issue #17 ) #36

Closed
wants to merge 1 commit into from

Conversation

ZoharLiran
Copy link
Contributor

Attempt to add guards to take care of issue 17
https://github.com/tvkhoa/react-tippy/issues/17

@ZoharLiran ZoharLiran changed the title Add guards Add guards ( Issue #17 ) Nov 19, 2017
@tuanthieu2993
Copy link
Contributor

Thanks @ZoharLiran for your contribution,

This is have been solved on v1.2.3, please upgrade to match with new updates.

Thanks

@ZoharLiran
Copy link
Contributor Author

Thank you for this new version. it indeed solved that error. but it seems like it is generating another error:
[getPopperElement]: Element passed as the argument does not exist in the instance

is this related to something you're working on already? or should I investigate further?

@ZoharLiran
Copy link
Contributor Author

UPDATE - removing the title prop when html prop is present helped with this latest one.

@tvkhoa
Copy link
Owner

tvkhoa commented Oct 23, 2019

Because of conflicting, I manually merge your commits to master and rebuild again. Thank you

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.

3 participants