fix(Tooltip): Prevent unnecessary setState calls

[grommett]

This PR prevents the handleClickOutside from calling setState if the ToolTip is already closed.

The reason for this check is that, at scale, these calls create real performance issues.

Our app has over 1,000 <Tooltip>s. Each of these Tooltips calls setState({open: false}). These unnecessary calls to setState makes toggling radio buttons and opening Modals very, very slow. Up to two seconds in some cases in our UI. After adding this check the UI behaves normally.

Performance screenshot below.

[asudoh]

LGTM 👍 - Thanks @grommett!

[stale]

We've marked this issue as stale because there hasn't been any activity for a couple of weeks. If there's no further activity on this issue in the next three days then we'll close it. You can keep the conversation going with just a short comment. Thanks for your contributions.

[stale]

We've marked this issue as stale because there hasn't been any activity for a couple of weeks. If there's no further activity on this issue in the next three days then we'll close it. You can keep the conversation going with just a short comment. Thanks for your contributions.

[stale]

We've marked this issue as stale because there hasn't been any activity for a couple of weeks. If there's no further activity on this issue in the next three days then we'll close it. You can keep the conversation going with just a short comment. Thanks for your contributions.

[emyarod]

looks good to me