tooltip trigger must be called twice after call to hide

[jehhynes]

#16014 introduced a bug where a link must be clicked twice to show the tooltip after calling hide. Fixed by setting inState variables to false in the hide() method.

[twbs-savage]

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: ef8bdd5f3231eee2445b28ca30975088bfac8bae
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/82203552

(Please note that this is a fully automated comment.)

[cvrebert]

You're presumably referring to bug #16732.

[cvrebert]

JavaScript changes must be accompanied by relevant unit test(s). See https://github.com/twbs/bootstrap/tree/master/js/tests#how-does-bootstraps-test-suite-work for info on how to add new unit test(s).

[jehhynes]

Yes, I am referring to bug #16732.

Unit test has been added.

[twbs-savage]

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: e4ca17151502cea933df1d11274d167395067d24
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/82637705

(Please note that this is a fully automated comment.)

[twbs-savage]

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED

Commit: 22f31a4
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/82957304

(Please note that this is a fully automated comment.)

[redbmk]

@jehhynes Thanks for the fix and tests! Looks like the build is failing because of some trailing whitespace, preventing them from proceeding with the pull request.

[cvrebert]

(Strictly speaking, this is currently blocked waiting for @fat to code review it.)

[redbmk]

Ah, I was just now getting around to looking at the bug and saw there was already this pull request. Looks like it's been here a couple weeks.

[gregblass]

+1 for the powers that be to release this! This is a pretty solid bug IMO.

[markbao]

@jehhynes, would you be able to update this for 4.0, or can I adapt this PR to submit a new one? inState no longer exists and seems to be replaced by _activeTrigger.

e.g. this patches the issue:

$('body').on('hidden.bs.popover', function (e) {
    $(e.target).data("bs.popover")._activeTrigger.click = false;
});

[jehhynes]

@markbao Please make this update and submit a new PR

[mdo]

Bootstrap 3 is no longer being officially developed or supported.

All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!

<3,
@mdo and team