Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Issue #3167 still present if using tooltip-popup-delay #3611

Closed
ntrifunovic opened this issue May 4, 2015 · 4 comments
Closed

Issue #3167 still present if using tooltip-popup-delay #3611

ntrifunovic opened this issue May 4, 2015 · 4 comments

Comments

@ntrifunovic
Copy link

Issue #3167, fixed in 87a3607, is still present if using tooltip-popup-delay with the tooltip.

Explanation: The tooltip is opened after the check is made to see if the tooltip is opened, when the button becomes disabled. This will lead to the same issue as described in #3167 .

@rvanbaalen
Copy link
Contributor

Updated plunkr: http://plnkr.co/edit/AUKz0kVzwMwlzZJDiHsr

@rvanbaalen
Copy link
Contributor

@ntrifunovic Can you confirm your issue is fixed in my plunkr?

I'll submit a PR for this if you confirm it's fixed.

@rvanbaalen rvanbaalen self-assigned this May 4, 2015
@rvanbaalen rvanbaalen modified the milestones: 0.13.x, 0.13.1 (Performance) May 4, 2015
@ntrifunovic
Copy link
Author

Yes.

          if (popupTimeout && val) {
              $timeout.cancel(popupTimeout);
          }

That seams to do the trick 😃

@rvanbaalen
Copy link
Contributor

Allright ill squeeze it in soon. Ill close this automatically when the PR lands.

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

No branches or pull requests

3 participants