Make compatible with RN 49+, refactored and added onHide and onShow props

[enahum]

• Refactored to be compatible with RN 49+
• Added a handler for didHide
• Added a flag to identify if there is a tooltip visible
• Added props for onHide and onShow
• Updated readme file

@chirag04 let me know if it is ok to merge and how can we do the update of the npm library ;)

[chirag04]

i would put this in the component and expose via ref. this.refs.tooltip.isToolTipVisible.

[enahum]

the idea is to use it as an import.

if you have multiple tooltip components and you want to know if there is at least one of those components showing the tooltip without looping though all of them.

does it make sense?

[chirag04]

if you have multiple tooltip components and you want to know if there is at least one of those components showing the tooltip without looping though all of them.

That sounds like a state that should live in your application then. It's little weird to keep this global export in a library.idk

[chirag04]

lgtm otherwise

[enahum]

@chirag04 agree, I removed the isToolTipVisible completely

[chirag04]

@enahum i guess update the readme also and merge.

[fungilation]

This is a much bigger refactor than my PR, which works for me up to and including RN 0.53. Looks like a nice cleanup, if everything still works the same!

[enahum]

@fungilation yeap everything works the same ;)

@chirag04 readme updated and merging...