Skip to content

Commit 1c127a6

Browse files
committed
fix(Popover): toggle twice to close
1 parent c77b0f3 commit 1c127a6

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

js/src/tooltip.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,6 @@ class Tooltip extends BaseComponent {
163163
return
164164
}
165165

166-
this._activeTrigger.click = !this._activeTrigger.click
167166
if (this._isShown()) {
168167
this._leave()
169168
return

js/tests/unit/popover.spec.js

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,26 @@ describe('Popover', () => {
5656
})
5757

5858
describe('show', () => {
59+
it('should toggle a popover after show', () => {
60+
return new Promise(resolve => {
61+
fixtureEl.innerHTML = '<a href="#" title="Popover" data-bs-content="https://twitter.com/getbootstrap">BS twitter</a>'
62+
63+
const popoverEl = fixtureEl.querySelector('a')
64+
const popover = new Popover(popoverEl)
65+
66+
popoverEl.addEventListener('shown.coreui.popover', () => {
67+
expect(document.querySelector('.popover')).not.toBeNull()
68+
popover.toggle()
69+
})
70+
popoverEl.addEventListener('hidden.coreui.popover', () => {
71+
expect(document.querySelector('.popover')).toBeNull()
72+
resolve()
73+
})
74+
75+
popover.show()
76+
})
77+
})
78+
5979
it('should show a popover', () => {
6080
return new Promise(resolve => {
6181
fixtureEl.innerHTML = '<a href="#" title="Popover" data-coreui-content="https://twitter.com/getbootstrap">BS twitter</a>'

0 commit comments

Comments
 (0)