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

$tooltipProvider.setTriggers with multiple events broken with v. 0.13.4 #4371

Closed
LeleDev opened this issue Sep 8, 2015 · 5 comments
Closed

Comments

@LeleDev
Copy link

LeleDev commented Sep 8, 2015

I have the following code in my app:

$tooltipProvider.setTriggers({'mouseenter':'mouseleave click'});

to make my tooltip close either clicking OR leaving the elements.
Since the last update, none of these two events does trigger a tooltip hide action. If instead I keep just one of the two events ('mouseleave' or 'click'), the tooltip behaves has expected. So I guess that something is broken regarding multiple events triggers support.

Thank you in advance

@wesleycho
Copy link
Contributor

Can you whip up a quick Plunker demonstrating this?

@wesleycho wesleycho added this to the 0.14.0 (Bootstrap 3.3) milestone Sep 8, 2015
@LeleDev
Copy link
Author

LeleDev commented Sep 9, 2015

@wesleycho
Copy link
Contributor

Thanks - I was able to fix this quickly thanks to that reproduction

@LeleDev
Copy link
Author

LeleDev commented Sep 9, 2015

Great job! Thank you!

@LeleDev LeleDev closed this as completed Sep 9, 2015
@wesleycho
Copy link
Contributor

Don't close this just yet, we'll close it when the PR is merged in :)

@wesleycho wesleycho reopened this Sep 9, 2015
jasonaden pushed a commit to deskfed/bootstrap that referenced this issue Jan 8, 2016
- Due to switch to raw `addEventListener`, changed to add and remove
  listeners to event triggers by iteration

Fixes angular-ui#4371
Closes angular-ui#4384
jasonaden pushed a commit to deskfed/bootstrap that referenced this issue Jan 8, 2016
- Due to switch to raw `addEventListener`, changed to add and remove
  listeners to event triggers by iteration

Fixes angular-ui#4371
Closes angular-ui#4384
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants