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

Fixes IE bug when empty data-tip. #190

Conversation

arnulfholm
Copy link

How to see the bug:

  1. Create two elements with data-tip attribute
  2. Add content to one of the data-tip attributes, and have en empty string in the other
  3. Hover over the one with content
  4. Exit the element
  5. Hover over the one without content

Result:
An empty tooltip will appear in IE 10 and 11 (probably others as well)
If exit the element without content, then enter again, all is ok.

Probable reason:
Since the content of the tooltip is not reset after exit, some logic probably thinks there is content, even though the new item does not have content.

How to see the bug:

1. Create two elements with data-tip attribute
2. Add content to one of the data-tip attributes, and have en empty string in the other
3. Hover over the one with content
4. Exit the element
5. Hover over the one without content

Result:
An empty tooltip will appear in IE 10 and 11 (probably others as well)
If exit the element without content, then enter again, all is ok.

Probable reason:
Since the content of the tooltip is not reset after exit, some logic probably thinks there is content, even though the new item does not have content.
@wwayne
Copy link
Collaborator

wwayne commented Sep 30, 2016

Thanks for exploring the problem, but I found it's little more complicated than what we thought. For example, the problem would be still there if we are using delayHide.

Finally I add a new state isEmptyTip to control this situation and sends a PR #201.

Anyway, thanks for your findings and PR.

@wwayne wwayne closed this Sep 30, 2016
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.

2 participants