namespace the events for popover/tooltip

[lookfirst]

namespace the events for popover/tooltip so that they can be cleanly removed.

references issue #3880

[travisbot]

This pull request passes (merged 2ee9b27 into 40ab928).

[lookfirst]

I wasn't sure what you wanted for the config key name... ns seemed short and sweet, but i'm open to changing it however you want.

[travisbot]

This pull request passes (merged 117f65d into 40ab928).

[fat]

Hey @lookfirst,

Thanks for opening this pull-request! Unfotunately, it looks like it fails to pass the tests neccessary for submitting to bootstrap. The following tests are currently failing:

• should always include a unit test if changing js files

For a full list of issue filing guidelines, please refer to the bootstrap issue filing guidelines.

thanks!

[lookfirst]

@fat umm... I did include tests.

[fat]

weird, sorry about that

[travisbot]

This pull request passes (merged 117f65d into 40ab928).

[lookfirst]

just realized that this could be combined into this.hide().off(this.options.ns)... if you like.

[fat]

yes pls

[fat]

any reason not to just always use the prefix .tooltip and .popover? Not super crazy about adding an option for that

[travisbot]

This pull request passes (merged 393f4a7 into 40ab928).

[lookfirst]

i'm not super crazy about the option either, but i could easily see a generic word like those being used by other plugins.

how about something a bit less generic like bs-popover/bs-tooltip?

[fat]

well... everywhere else we just use "modal" or "button". I don't think it's too big of a deal. If you have two different tooltip plugins instantiated on the same element you probably have bigger problems ;)

In the future we'll probably namespace everything with bootstrap (data-attrs and events - but that will be a 3.0.0 thing, because it's breaks backwards compatibility)

[lookfirst]

ok, i'll hardcode it then.

[fat]

cool thanks :)

[lookfirst]

Oh, looking at that more closely now, I know why I did that... popover extends tooltip and tooltip.init() is where the .on() is... I needed to define the name of the plugin somewhere...

[fat]

this.type will equal 'popover' or 'tooltip' - it's assigned in the init method

[travisbot]

This pull request passes (merged d76c899 into 40ab928).

[lookfirst]

ah, missed that. should be good now. all tests pass.

[fat]

great - thanks man!