diff --git a/js/src/tooltip.js b/js/src/tooltip.js index 92d455349336..097477f7a1a8 100644 --- a/js/src/tooltip.js +++ b/js/src/tooltip.js @@ -160,7 +160,6 @@ class Tooltip extends BaseComponent { return } - this._activeTrigger.click = !this._activeTrigger.click if (this._isShown()) { this._leave() return diff --git a/js/tests/unit/popover.spec.js b/js/tests/unit/popover.spec.js index ba38ebe06629..1c81ba79279c 100644 --- a/js/tests/unit/popover.spec.js +++ b/js/tests/unit/popover.spec.js @@ -56,6 +56,26 @@ describe('Popover', () => { }) describe('show', () => { + it('should toggle a popover after show', () => { + return new Promise(resolve => { + fixtureEl.innerHTML = 'BS twitter' + + const popoverEl = fixtureEl.querySelector('a') + const popover = new Popover(popoverEl) + + popoverEl.addEventListener('shown.bs.popover', () => { + expect(document.querySelector('.popover')).not.toBeNull() + popover.toggle() + }) + popoverEl.addEventListener('hidden.bs.popover', () => { + expect(document.querySelector('.popover')).toBeNull() + resolve() + }) + + popover.show() + }) + }) + it('should show a popover', () => { return new Promise(resolve => { fixtureEl.innerHTML = 'BS twitter' diff --git a/site/content/docs/5.3/components/modal.md b/site/content/docs/5.3/components/modal.md index e00931e8162e..9f42f5aba358 100644 --- a/site/content/docs/5.3/components/modal.md +++ b/site/content/docs/5.3/components/modal.md @@ -201,7 +201,7 @@ When modals become too long for the user's viewport or device, they scroll indep
This is some placeholder content to show the scrolling behavior for modals. Instead of repeating the text in the modal, we use an inline style to set a minimum height, thereby extending the length of the overall modal and demonstrating the overflow scrolling. When content becomes longer than the height of the viewport, scrolling will move the modal as needed.