Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-tooltip: Support hide-* and show-* attributes #2386

Closed
epelc opened this issue Apr 17, 2015 · 2 comments
Closed

md-tooltip: Support hide-* and show-* attributes #2386

epelc opened this issue Apr 17, 2015 · 2 comments

Comments

@epelc
Copy link
Contributor

epelc commented Apr 17, 2015

I get the following error when hovering over something that has a disabled tooltip which is using one of the hide or show classes. Also please note this error only happens when you hide the tooltip for whatever you are hovering over.

In my case I'm using

<md-tooltip hide show-sm>
      <span ng-bind="..."></span>
</md-tooltip>
"TypeError: Cannot read property 'scrollHeight' of null
    at positionTooltip (http://dev.*******.com/app/vendor.js:30029:65)
    at showTooltip (http://dev.*******.com/app/vendor.js:30003:8)
    at Object.fn (http://dev.*******.com/app/vendor.js:29942:25)
    at n.$get.n.$digest (https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js:123:356)
    at n.$get.n.$apply (https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js:126:293)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js:139:3
    at e (https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js:40:74)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js:44:155"

Note this just started with 0.9.0-rc1 it worked fine in 0.8.3

@epelc epelc changed the title md-tooltip: Error when using with hide-* or show-* md-tooltip: Support hide-* and show-* attributes Apr 22, 2015
@Lenninlasd
Copy link

Hello this issue, appears again in v0.11.1

@topherfangio
Copy link
Contributor

@lasdjs Can you please open a new issue with a Codepen that demonstrates it? Here are some links you can use:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants