Bug: Maximum update depth exceeded

[kukchanka]

I believe there is an issue with getPosition fucntion which is used to update tooltip position when it's outside the client widnow.

There are situations with floating tooltip (e.g. when we move cursor to the bottom of the page) when it cannot be displayed, so it tries to calculate a new desiredPlace and rerenders the component, but after rerendering getPosition finds out, that a previously calculated desiredPlace doesn't allow to render tooltip inside the client window, so it calculates a new one, which turns out to be the one we started with. It starts switching back and forth and "maximum update depth exceeded" error occurs.

To demostrate this, I added a pair console.logs inside getPosition and moved the cursor over tooltiped element:

This is what I got in my console just right before "Maximum update depth exceeded" error:

[kukchanka]

Created a pull-request fixing this issue: #528

[Zemelia]

Issue still exists, it was returned after fix in #574

And in my case I see the problem, when based on position we add some extra styling to tooltip, i.e. left or top borders. So it goes into loop when trying to "updatePosition".

[pankajgandhi1010]

UpdatePosition() shows Error : Maximum update depth exceeded.