Skip to content

Commit

Permalink
Merge pull request #15425 from twbs/regression-test-14244
Browse files Browse the repository at this point in the history
Add regression test for #14244.
  • Loading branch information
cvrebert committed Dec 30, 2014
2 parents e4c7a92 + 27bfef5 commit 9dc3cf7
Showing 1 changed file with 38 additions and 0 deletions.
38 changes: 38 additions & 0 deletions js/tests/unit/popover.js
Original file line number Diff line number Diff line change
Expand Up @@ -217,4 +217,42 @@ $(function () {
$div.find('a.second').click()
equal($('.popover').length, 0, 'second popover removed')
})

test('should detach popover content rather than removing it so that event handlers are left intact', function () {
var $content = $('<div class="content-with-handler"><a class="btn btn-warning">Button with event handler</a></div>').appendTo('#qunit-fixture')

var handlerCalled = false;
$('.content-with-handler .btn').click(function () {
handlerCalled = true
});

var $div = $('<div><a href="#">Show popover</a></div>')
.appendTo('#qunit-fixture')
.bootstrapPopover({
html: true,
trigger: 'manual',
container: 'body',
content: function () {
return $content;
}
})

stop()
$div
.one('shown.bs.popover', function () {
$div
.one('hidden.bs.popover', function () {
$div
.one('shown.bs.popover', function () {
$('.content-with-handler .btn').click()
$div.bootstrapPopover('destroy')
ok(handlerCalled, 'content\'s event handler still present')
start()
})
.bootstrapPopover('show')
})
.bootstrapPopover('hide')
})
.bootstrapPopover('show')
})
})

0 comments on commit 9dc3cf7

Please sign in to comment.