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

feat(tooltip): remove unnecessary $digest #4151

Closed
wants to merge 1 commit into from

Conversation

wesleycho
Copy link
Contributor

  • Remove unnecessary call to $digest

Seems to work perfectly fine here.

- Remove unnecessary call to `$digest`
@icfantv
Copy link
Contributor

icfantv commented Aug 8, 2015

👍

@wesleycho wesleycho closed this in 901a7c6 Aug 8, 2015
@wesleycho wesleycho deleted the perf/tooltip-digest branch August 8, 2015 16:41
@djeremh
Copy link

djeremh commented Aug 8, 2015

In your plunkr, the custom trigger tooltip doesn't work.

It seems refering to the commit 5726e3e.

@wesleycho
Copy link
Contributor Author

@djeremh it does actually work - you have to scroll to the right though due to the window size allocated on Plunker.

@djeremh
Copy link

djeremh commented Aug 8, 2015

tricky, my bad on this.

@wesleycho
Copy link
Contributor Author

Not a problem, I've had to do a double take multiple times over various issues too :) .

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.

3 participants