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

feat(tooltip): Setting to show only single popup #751

Closed
wants to merge 1 commit into from
Closed

feat(tooltip): Setting to show only single popup #751

wants to merge 1 commit into from

Conversation

vchatterji
Copy link

Can be invoked using:
$tooltipProvider.options({ popoverMode: 'single'});

The code is big, but it's mostly comments.

I think it would be good idea to do this globally (through tooltipProvider). The popover-group
directive is also an interesting idea, but this is much simpler to use
for the end user. No?

@vchatterji
Copy link
Author

See issues:
#618
and
#590

This feature allows for a single popover being displayed. Any new popover will close any older popovers. Clicking on body will also close the popover. The behaviour is optional and by default the popover will retain its existing multiple display functionality.

This feature can be turned on via:
$tooltipProvider.options({ popoverMode: 'single'});

I am resubmitting this pull request after closing the last one. Code is cleaned up and in a single commit. The old pull request was:
#749

@vchatterji
Copy link
Author

Plunker with this code:
http://plnkr.co/edit/7XHqwY?p=preview

Can be invoked using:
$tooltipProvider.options({ popoverMode: 'single'});

The code is big, but it's mostly comments.

I think it would be good idea to do this globally. The popovergroup
directive is also an interesting idea, but this is much simpler to use
for the end user. No?
@vchatterji
Copy link
Author

Sorry for closing the previous pull request. I am still learning the git way.

Cheers.

@vchatterji
Copy link
Author

Still getting to know github. Closing this request in favour of two new pull request one which implements this as a $tooltipProvider option and the other one as a directive.

@vchatterji vchatterji closed this Aug 5, 2013
@tmundt
Copy link

tmundt commented May 13, 2014

Was this included in the last UI bootstrap version 0.11.0?

@mark47
Copy link

mark47 commented Jun 27, 2014

Doesn't appear to be in 0.11.0. It worked great when I did replace the existing code with @vchatterji implementation. Thanks!

codedogfish pushed a commit to codedogfish/angular-ui-bootstrap that referenced this pull request Sep 15, 2015
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