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

fix(tooltip): fix binding to multiple triggers #4384

Closed

Conversation

wesleycho
Copy link
Contributor

  • Due to switch to raw addEventListener, changed to add and remove
    listeners to event triggers by iteration

Here is the fix in action.

This fixes #4371

- Due to switch to raw `addEventListener`, changed to add and remove
  listeners to event triggers by iteration
@icfantv
Copy link
Contributor

icfantv commented Sep 9, 2015

👍

@icfantv
Copy link
Contributor

icfantv commented Sep 9, 2015

I think this may also fix #4364.

@wesleycho wesleycho closed this in d6cda93 Sep 10, 2015
@wesleycho wesleycho deleted the fix/tooltip-multiple-triggers branch September 10, 2015 10:32
jasonaden pushed a commit to deskfed/bootstrap that referenced this pull request 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 pull request 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 this pull request may close these issues.

$tooltipProvider.setTriggers with multiple events broken with v. 0.13.4
2 participants