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

Conversation

maxisam
Copy link

@maxisam maxisam commented Jul 11, 2013

@maxisam
Copy link
Author

maxisam commented Jul 11, 2013

I think if we make isOpen to link with tooltip scope with '=' instead of '&' , we can get rid of show / hide I put on tooltip
Here is the example how I think http://plnkr.co/edit/qXuuos55FROxRxSRZFL9?p=preview

I don't know why isOpen use '&' now, so I don't change that part.

@maxisam maxisam closed this Jul 11, 2013
@maxisam maxisam reopened this Jul 11, 2013
@pkozlowski-opensource
Copy link
Member

@maxisam thnx for this PR, I know what you are trying to achieve but your changes as-is are breaking the build so can't be merged. Going to close this one for now, feel free to open another PR when you've got a complete fix.

Even if this one is not getting merged we really do appreciate your effort!

@maxisam
Copy link
Author

maxisam commented Jul 27, 2013

I understand. I am sorry I don't have time to provide a better solution
right now. It is just for someone who wants to have a walk around and
doesn't care about side effect.

On Sat, Jul 27, 2013 at 12:14 PM, Pawel Kozlowski
notifications@github.comwrote:

@maxisam https://github.com/maxisam thnx for this PR, I know what you
are trying to achieve but your changes as-is are breaking the build so
can't be merged. Going to close this one for now, feel free to open another
PR when you've got a complete fix.

Even if this one is not getting merged we really do appreciate your effort!


Reply to this email directly or view it on GitHubhttps://github.com//pull/654#issuecomment-21668913
.

@akhilesh84
Copy link

Hi - Did we ever get a closure on this one?

codedogfish pushed a commit to codedogfish/angular-ui-bootstrap that referenced this pull request Sep 15, 2015
fix(templates): use ng-bind instead of ng-bind-html
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 this pull request may close these issues.

3 participants